-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Datepicker design improvements #5691
Comments
@pixelipo I am not one of the designers, but: Changes: Since dates never go into the triple digits, i don't understand why they are right-aligned. Rationale: Distinguishing the current date from the selection can be done by having both current month, and current day, displayed in blue. Months on the side: Time field |
Agree with both of your suggestions @pixelipo @comradekingu. Just small additions:
Here’s a a rough mockup (the circles should be round of course): |
Me again: Changes: Rationale: Ideally, worst case:
< des* > < current year* > Actual result: < ↓ > < current year* > @jancborchardt (Got ninja'd.) Looks much cleaner. If not a bit low on contrast. |
@jancborchardt Something like this? Changes: Rationale: Edit: Bubbles looked better. Might be able to get away with just underline for months and year. Would be interesting to colour the >'s or <'s pointing to respectively a later or previous current month or year. |
@comradekingu selected date should be more visible than today - the opposite is try in your example. Separation of month and year would be nice, but that is not as simple to do as all the other changes so far mentioned |
This is a 3-step approach, and we need to make sure not to bunch it up too much or over-discuss it or we risk not getting anything done. ;)
|
Steps 1 and 2 ready for review in #5713 |
@jancborchardt in your design it will look better if you had more margins in all the sides. |
This is a copy of nextcloud/deck#205 because the same datepicker is used in many Nexcloud apps, inlcuding nextcloud/files and @nextcloud/calendar
It would be useful to highlight "today" in the datepicker. It's a small fix, but some feedback from @nextcloud/designers is needed first.
Here's my suggestion:
However, two things must be considered as well:
datepicker doesn't look very much integrated into the general UI (buttons, icons, colours, fonts...) - do we want to make a more general overhaul of it? Here's an example of what it might look like:
Files app by default disables (in "set expiration date") dates older than "tomorrow", so it might not be weird to highlight "today". This is how my suggestion would appear over there:
The text was updated successfully, but these errors were encountered: