-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[pickers] Remove clear button in tab sequence #14587
Comments
Seems like a valid change and with good reasoning. I did add this to the board so we can sicuss further in the grooming! 👍🏼 |
In the autocomplete, the The only consequence it will have is that people should have the possibility to open the calendar by keyboard whilst focusing on the field. If I'm not mistaken, if you press |
@GBvanDam for a widget with the TL/DR: Yes, using Space to open the Picker makes sense, but no, I disagree with removing the open button from the tab sequence. 😉 |
This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue. |
@LukasTy completely agree! Great to hear that you at least thought about it, and if I understand you correctly the open on space will be added, which is great! Thanks for your support! |
Steps to reproduce
Link to live example: (required)
Steps:
Screen.Recording.2024-09-11.at.20.20.55.mov
Current behavior
The clear button is in the Tab sequence.
Expected behavior
The clear button is not in the tab sequence.
Context
Started to discuss this in #13289 (comment).
In the autocomplete we ruled to not have it in the tab sequence:
It would make sense to me to do the same here for the sake of being simple to navigate inside a complex form with the keyboard. The date picker seems simpler to clear than an autocomplete:
More on this: https://scottaohara.github.io/clear-text-field-button/.
Your environment
npx @mui/envinfo
Search keywords: -
The text was updated successfully, but these errors were encountered: