-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
fix(datepicker): cursor fixes for disabled dates & weeks in datepicker #4905
Conversation
Codecov Report
@@ Coverage Diff @@
## development #4905 +/- ##
===============================================
+ Coverage 74.51% 74.59% +0.08%
===============================================
Files 273 273
Lines 8365 8365
Branches 1594 1594
===============================================
+ Hits 6233 6240 +7
+ Misses 1696 1688 -8
- Partials 436 437 +1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## development #4905 +/- ##
==============================================
+ Coverage 74.63% 74.7% +0.07%
==============================================
Files 276 276
Lines 8385 8385
Branches 1594 1594
==============================================
+ Hits 6258 6264 +6
+ Misses 1687 1680 -7
- Partials 440 441 +1
Continue to review full report at Codecov.
|
Nice to have: remove cursor-pointer from weekdays on the left column also |
@daniloff200 Thank you! Well done. Tested in Chrome, FF, IE - looks pretty good) |
PR Checklist
Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.