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

[EuiDatePickerRange] Using minDate and maxDate breaks the year selection popover UI #5058

Closed
ashokaditya opened this issue Aug 20, 2021 · 4 comments · Fixed by #5069
Closed
Labels

Comments

@ashokaditya
Copy link
Member

ashokaditya commented Aug 20, 2021

If you restrict endDateControl selections based on startDateControl value and vice versa, the popover UI breaks as sometimes years shown are not in multiples of 3.

To reproduce:

  1. Restrict end-date to be selected between start-date value and now, using minDate and maxDate.
  2. Restrict start date to be not greater than the end-date using maxDate.
  3. Click on year selection of either start or end-date inputs.
  4. Observe that the popover shows extra spaced out values, in some cases a blank pop over.

clip
date-picker-bug

@ashokaditya ashokaditya changed the title [EuiDatePicker] Using minDate and maxDate breaks the year selection popover UI [EuiDatePickerRange] Using minDate and maxDate breaks the year selection popover UI Aug 20, 2021
@thompsongl
Copy link
Contributor

I believe this is the same thing as #4219 but let's keep both open until verified.

@ashokaditya
Copy link
Member Author

@thompsongl indeed. I searched for [EuiDatePickerRange] issues and didn't find one so created this one.

@ashokaditya
Copy link
Member Author

@thompsongl is it possible to list out all the legit bugs within https://elastic.github.io/ somehow. It could be like a list of bug links for each components doc page. It would save a lot of time not creating bugs that have already been reported. 🤔

@thompsongl
Copy link
Contributor

Also related: #2816

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