Skip to content

Post time range input#3960

Closed
Nutomic wants to merge 10 commits into
mainfrom
post-time-range-input
Closed

Post time range input#3960
Nutomic wants to merge 10 commits into
mainfrom
post-time-range-input

Conversation

@Nutomic
Copy link
Copy Markdown
Member

@Nutomic Nutomic commented Mar 10, 2026

This one is much more intuitive to use:
Screenshot_20260311_111819

TODO:

  • Add more time values (those which are currently available in 0.19)
  • Hide time input if current sort is Old or New
  • Avoid using vertical space for label (tooltip could work, but doesnt support live update)
  • Reorder buttons
  • Same input for community feed
  • Put the existing TimeIntervalFilter in the expanded menu, or otherwise delete it

Requires #3944

@Nutomic Nutomic marked this pull request as ready for review March 10, 2026 15:07
@dessalines
Copy link
Copy Markdown
Member

I really don't like this, it takes up way too much space, and loses all the custom flexibility. Please keep it the way it was.

@Nutomic
Copy link
Copy Markdown
Member Author

Nutomic commented Mar 12, 2026

It only takes 24px of vertical space, while the filter buttons which are now hidden by default take 31px. Also this has much better usability especially on mobile. I added back TimeIntervalFilter now (without presets), so you can also set custom intervals. At the moment both time inputs are not synchronized yet.

Screenshot_20260312_110044

@dessalines
Copy link
Copy Markdown
Member

dessalines commented Mar 12, 2026

I'm still not a big fan of it, and I'd like to get other input from @SleeplessOne1917 or @matc-pub . A bar like this (especially where the intervals are random), tells you nothing by glancing at it (compare that with the existing dropdown telling you 5 weeks). Every UI element should be immediately comprehensible as to what its showing you, and a bar tells you nothing (unless its like a percentage or something)

I spose I'd be fine with it as long as its hidden by default with the expand dropdown, and the time interval filter dropdown stays.

@Nutomic
Copy link
Copy Markdown
Member Author

Nutomic commented Mar 12, 2026

There is a label with the current value at the top right, that could be moved somewhere closer to the slider. It would also be possible to add step markers and labels, something like this:
Screenshot_20260312_160617

https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Elements/input/range#examples

@dessalines
Copy link
Copy Markdown
Member

That's kinda why you don't see hardly any UIs using these range inputs nowadays, because they don't look good. Dropdowns are far better.

@dessalines
Copy link
Copy Markdown
Member

This can probably be closed due to matc's PR.

@Nutomic Nutomic closed this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants