[BUG] mouse hover effect on range picker

[metalmaninfr]

Expected Behavior

on range date picker, on 1st onClick on date, the mouse hover effect should stay on the day we clicked before mouseLeave

Current Behavior

When I click on a day, a selection of days has a mouse hover effect

Possible Solution

no idea

Your Environment

Mac os 11.4 - Chrome
React 17.1
next 10.3

Additional Comments

https://gyazo.com/fc2916dd34f3dcd120ae1c9874764515

[coling]

I've seen this too. I'll try and take a look sometime soon if @samsam-ahmadi doesn't beat me to it

[coling]

Just FYI, I fixed this in a commit I've just included in an open pull request #22
I can split this out (as it's arguably unrelated) but otherwise it should get into a release soon enough 😃

[samsam-ahmadi]

@coling @metalmaninfr, Thank you so much, Sorry I have feeling bad. I will check them ASAP.

[coling]

@coling @metalmaninfr, Thank you so much, Sorry I have feeling bad. I will check them ASAP.

There is no rush on my part. Rest up and feel better :-)

[z-Amiryan]

I have same problem with the RangePicker component when I try to setState dates(from, to), mouse hover effect disappears.

[coling]

I think this is fixed now @samsam-ahmadi as I did fix up the class application in the recent pull request you just merged. You may want to tidy it up and make it clearer/better tho' 😄

[samsam-ahmadi]

Thanks @coling. ​
I just merged it to test it here; I just understand we have a problem with the custom title of weeks. If you change the startOfWeek here, you can see the titles have a problem.

https://killthejs.com/react-trip-date/?path=/story/range-picker-component--custom-components-header-titles

[samsam-ahmadi]

I fixed it with this commit

And new release is available: @1.7.0

[coling]

Thanks @coling. ​
I just merged it to test it here; I just understand we have a problem with the custom title of weeks. If you change the startOfWeek here, you can see the titles have a problem.

https://killthejs.com/react-trip-date/?path=/story/range-picker-component--custom-components-header-titles

🤦🏼 I mustn't have tested that properly.... Sorry 😢 and thanks for the fix 😄

[metalmaninfr]

I'm still having the same problems, as I am resetting the "start-date" and the "end_date" to null, i think the getEndDateForClasses is not triggering, i think it should be called on onClick

https://gyazo.com/affe1abcf9dc0dfc020d0fb0abcfaf80

[samsam-ahmadi]

@metalmaninfr Would you please reproduce it in the codesandbox ?

[metalmaninfr]

https://codesandbox.io/s/angry-stonebraker-1wxr4

[samsam-ahmadi]

@metalmaninfr Please check this version [email protected]

[metalmaninfr]

seems to work, thank you :)

[samsam-ahmadi]

@metalmaninfr Great. I closed this issue, if you have any other issue or ideas feel free to open issue or PR.

[z-Amiryan]

thank you for fixing this bug))