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

EuiSuperDatePicker seems to refresh pages unexpectedly #7503

Closed
BryceRyan opened this issue Feb 2, 2024 · 1 comment · Fixed by #7504
Closed

EuiSuperDatePicker seems to refresh pages unexpectedly #7503

BryceRyan opened this issue Feb 2, 2024 · 1 comment · Fixed by #7504
Assignees
Labels

Comments

@BryceRyan
Copy link

Describe the bug
When wrapped in an EuiForm with the component set to form, the EuiSuperDatePicker will refresh the page while you are setting the date fields

Impact and severity
Low

Environment and versions
Replicated in provided Code Sandbox on EUI's website EUI version 92.2.1

To Reproduce
Steps to reproduce the behavior:

  1. Wrap an EuiSuperDatePicker with an EuiForm with component='form' set
  2. Select the EuiSuperDatePicker to get the start date -> end date to appear with a popover open on the start
  3. Click on the start or end buttons (as if to close the start popover or open the end popover)
  4. Notice that the page refreshes

Expected behavior
When the start or end date picker popovers close, the page should not refresh. It should instead continue on as normal.

Minimum reproducible sandbox
sandbox

Additional context
Did not occur when component was not set to 'form'

@BryceRyan BryceRyan added bug ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible labels Feb 2, 2024
@cee-chen
Copy link
Contributor

cee-chen commented Feb 2, 2024

Oh jeez haha, how did this go unnoticed for over 5 years? 😭 Thanks so much for catching this Bryce!

@cee-chen cee-chen removed the ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible label Feb 2, 2024
@cee-chen cee-chen self-assigned this Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants