components icon indicating copy to clipboard operation
components copied to clipboard

Remove calendarHasFocus from date-range-picker/calendar as unneeded

Open zshakeel-amazon opened this issue 3 years ago • 1 comments

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

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.

zshakeel-amazon avatar Aug 04 '22 13:08 zshakeel-amazon

Codecov Report

Merging #121 (55f4496) into main (2743cbd) will decrease coverage by 0.00%. The diff coverage is 100.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.

codecov[bot] avatar Aug 04 '22 13:08 codecov[bot]