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

No class to identify start or end of the date range, while hovering, on multi-dates if one date is selected #809

Closed
Turito opened this issue Apr 5, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@Turito
Copy link

Turito commented Apr 5, 2024

Describe the bug
Since v8.3.0 (I think due to 3ce0e0a) if you use the multi-dates mode then it's no longer possible, after selecting a date, there is no class that identifies the start or end of the date range to have custom styles for hovering.

In v8.3.0, after selecting a day then:

  • Class .dp__date_hover is added on all enabled days.
  • Class .dp__date_hover_start is added on all days disabled and less than the selected date.
  • Class .dp__date_hover_end is added on all days disabled and higher than the selected date.

In v8.2.0, after selecting a day then:

  • Class .dp__date_hover_start was added on all days less than the selected date.
  • Class .dp__date_hover_end was added on all days higher than the selected date.

To Reproduce
Steps to reproduce the behavior:

  1. Go to this demo vue-datepicker@v8.3.2
  2. Click on a random date to select it.
  3. Hover on another non-selected date and see that there is no class that identifies the start or end of the date range

Expected behavior
The expected behavior would be the same of the v8.2.0

  • Class .dp__date_hover_start will be added on all days less than the selected date.
  • Class .dp__date_hover_end will be added to every day after the selected date.

Screenshots
v8.3.0
Screenshot 2024-04-05 at 11 08 24

v8.2.0
Screenshot 2024-04-05 at 10 40 37

Desktop & mobile (please complete the following information):

  • Browser: Chrome
  • Library version: v8.3.2
@Turito Turito added awaiting triage The issue is not reviewed by the maintainers bug Something isn't working labels Apr 5, 2024
@Jasenkoo Jasenkoo removed the awaiting triage The issue is not reviewed by the maintainers label Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants