fix: #4581 and #4560 selected and preSelection with showWeekPicker

[yuki0410-dev]

Description

Linked issue: close #4581,#4560
Problem
See issue #4581 and #4560

Changes
As per comment #4581, I have made the following changes to the showWeekPicker case.

• Remove processing to correct date and preSelection to the first day of the week.
• Changed the passing of date and preSelection from Month component to Week component so that they are corrected to the first day of the week before passing them to the Month component.

For #4560, a null selected was being passed to getStartOfWeek. This time, the processing timing of getStartOfWeek has been moved to Month, so that null is no longer included.

Screenshots

To reviewers

Contribution checklist

• I have followed the contributing guidelines.
• I have added sufficient test coverage for my changes.
• I have formatted my code with Prettier and checked for linting issues with ESLint for code readability.

[pullrequest]

✅ This pull request was sent to the PullRequest network.

---

@yuki0410-dev you can click here to see the review status or cancel the code review job.

[pullrequest]

PullRequest Breakdown

Reviewable lines of change

+ 21
- 36

95% JavaScript
5% JavaScript (tests)

Type of change

Fix - These changes are likely to be fixing a bug or issue.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.94%. Comparing base (a044048) to head (b07608e).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4601      +/-   ##
==========================================
+ Coverage   96.90%   96.94%   +0.03%     
==========================================
  Files          28       28              
  Lines        2583     2582       -1     
  Branches     1078     1081       +3     
==========================================
  Hits         2503     2503              
+ Misses         80       79       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pullrequest]

This looks good to me, I see no issue here.

Simon

---

Reviewed with ❤️ by PullRequest