-
Notifications
You must be signed in to change notification settings - Fork 394
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
[FluentCalendar] Use provided Culture for calendar operations #1862
[FluentCalendar] Use provided Culture for calendar operations #1862
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've already posted a few comments (first quick check).
I'm afraid of the number of "small" changes in the code base.
E.g. Year = year.GetDay(_calendar.Culture) == 1 && year.GetMonth(_calendar.Culture) == 1 ? year : year.StartOfYear(_calendar.Culture);
Could you add unit tests to validate your changes and new features?
examples/Demo/Shared/Pages/DateTimes/Examples/DatePickerDefault.razor
Outdated
Show resolved
Hide resolved
@microsoft-github-policy-service agree |
I've added some tests to ensure correct functionality by using Culture. And one thing more, I noticed that |
970b988
to
77ecdc2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job. Thanks.
Pull Request
📖 Description
As reported in issue #1854, FluentCalendar (and FluentDatePicker) didn't use Culture for calendar operations and calculations.
It's now fixed, and the calendar now supports every culture correctly.
🎫 Issues
#1854
👩💻 Reviewer Notes
📑 Test Plan
✅ Checklist
General
Component-specific