Fix with date range selected in st.date_input, hovered dates should change color #11223
Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
kmcgrady
left a comment
There was a problem hiding this comment.
Hey @Bernardo1008 . Apologies for the delay. I've been on vacation and just a bit overwhelmed. Appreciate your patience.
Things look overall well. Just had a question, and I think all you need to do is to is format it using prettier so that everything passes. Let me know if there's anything else I can do.
There was a problem hiding this comment.
I just tried not using the !important, and everything seems to work fine. Did you find a specific reason here that it is necessary?
afe4cef to
b9890ce
Compare
Improved the DateInput component by applying custom hover styles for dates that fall within a selected range, specifically for the light theme. Enhances visual clarity and user experience when selecting date ranges.
b9890ce to
3847527
Compare
|
Thanks @Bernardo1008 I made the changes and verified it worked and merged. Thanks for the contribution! |
Describe your changes
Improved the DateInput component by applying custom hover styles for dates that fall within a selected range, being a unique problem for the light theme. Enhances visual clarity and user experience when selecting date ranges.
GitHub Issue Link (if applicable)
fixes #10929
Testing Plan
No additional testing is needed, as the problem is fairly simple and involves only a color change in a specific area.
Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.