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

[DateRangePicker] Change open trigger to onMouseDown #10110

Open
LukasTy opened this issue Aug 22, 2023 · 1 comment
Open

[DateRangePicker] Change open trigger to onMouseDown #10110

LukasTy opened this issue Aug 22, 2023 · 1 comment
Labels
component: DateRangePicker The React component. component: pickers This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature on hold There is a blocker, we need to wait

Comments

@LukasTy
Copy link
Member

LukasTy commented Aug 22, 2023

Created from #9386 (comment)

Range pickers do open when clicked on the input element rather than the open button as the regular pickers.
The input element has its own behavior when it is moused down (the input focuses and the active section is selected), but the opening of the picker happens only when the click action is finished on the input.

The proposal is to use onMouseDown as the trigger to opening the range picker popper to increase the responsiveness and avoid this double effect of one behavior when input is moused down and another when the click is completed.

@LukasTy LukasTy added component: pickers This is the name of the generic UI component, not the React module! component: DateRangePicker The React component. enhancement This is not a bug, nor a new feature labels Aug 22, 2023
@LukasTy
Copy link
Member Author

LukasTy commented Jan 7, 2025

#14786 should remove this problem from the new default variant - single input range field.
Re-groom this issue after #14786 is done to assess the state with multi-input range field.

@LukasTy LukasTy added the on hold There is a blocker, we need to wait label Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: DateRangePicker The React component. component: pickers This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature on hold There is a blocker, we need to wait
Projects
None yet
Development

No branches or pull requests

1 participant