Skip to content
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

FEATURE: Remove separate "apply"-Button from DateTime Editor #3421

Closed
mficzel opened this issue Mar 9, 2023 · 1 comment · Fixed by #3736
Closed

FEATURE: Remove separate "apply"-Button from DateTime Editor #3421

mficzel opened this issue Mar 9, 2023 · 1 comment · Fixed by #3736
Labels
Feature Label to mark the change as feature UI & UX

Comments

@mficzel
Copy link
Member

mficzel commented Mar 9, 2023

Description

The "Apply" Button of the DateTime Editor is often not visible when the datetime editor ist in the lower half of the screen. This leads to confusion when editors choose a date but the Inspector apply button stays inactive.

Also it it a questionable ui-solution to have to push to different apply buttons in a row to actually select a date.

This could be nicely combined with #3394

Expected behavior

It would be great if the apply button at the datepicker would be removed entirely and the date would be set once the user sets a date (date-mode) or a (time).

Main Problem tody is that apply does two things: 1. Apply value, 2. Close DatePicker. If we find a different rule for closing the datepicker after editing apllying to values once they are slected should feel natural to editors.

[What actually happened]

Affected Versions

UI: since ever

@mficzel mficzel changed the title FEATURE: Remove "Apply"-Button from DateTime Editor FEATURE: Remove separate "apply"-Button from DateTime Editor Mar 9, 2023
@crydotsnake crydotsnake added UI & UX Feature Label to mark the change as feature labels Mar 9, 2023
@ahaeslich ahaeslich assigned ahaeslich and Alvadda and unassigned ahaeslich Mar 11, 2024
@Alvadda Alvadda removed their assignment Mar 13, 2024
@ahaeslich
Copy link
Member

Thx @Alvadda for your contribution 💖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Label to mark the change as feature UI & UX
Projects
No open projects
Status: Done
4 participants