components
components copied to clipboard
Remove calendarHasFocus from date-range-picker/calendar as unneeded
Description
Refactor date picker calendar by removing unneeded variable.
How has this been tested?
Existing tests -- no behaviour changes expected
Documentation changes
[Do the changes include any API documentation changes?]
- [ ] Yes, this change contains documentation changes.
- [ ] No.
Related Links
[Attach any related links/pull request for this change]
Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness
- [ ] Changes are backward-compatible if not indicated, see
CONTRIBUTING.md. - [ ] Changes do not include unsupported browser features, see
CONTRIBUTING.md. - [ ] Changes were manually tested for accessibility, see accessibility guidelines.
Security
- [ ] If the code handles URLs: all URLs are validated through the
checkSafeUrlfunction.
Testing
- [ ] Changes are covered with new/existing unit tests?
- [ ] Changes are covered with new/existing integration tests?
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Codecov Report
Merging #121 (55f4496) into main (2743cbd) will decrease coverage by
0.00%. The diff coverage is100.00%.
@@ Coverage Diff @@
## main #121 +/- ##
==========================================
- Coverage 92.38% 92.38% -0.01%
==========================================
Files 536 536
Lines 15473 15471 -2
Branches 4254 4253 -1
==========================================
- Hits 14295 14293 -2
Misses 1095 1095
Partials 83 83
| Impacted Files | Coverage Δ | |
|---|---|---|
| src/date-range-picker/calendar/header/index.tsx | 100.00% <ø> (ø) |
|
| src/date-range-picker/calendar/index.tsx | 79.84% <ø> (ø) |
|
| ...date-range-picker/calendar/header/button/index.tsx | 100.00% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.