-
Notifications
You must be signed in to change notification settings - Fork 179
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
Respect first day of the week setting #11651
Comments
Oh good call. Looks like there's no explicit setting for first day of the week (wojtekmaj/react-calendar#10), but only a const toCalendarType = (weekStartDay) =>
weekStartDay === 0 ? 'US' : 'ISO 8601'
} Maybe that works? |
There is a PR on the gutenberg side - WordPress/gutenberg#41648 |
Gutenberg uses a different library for the date picker, so what works for them probably does not work for us. |
I know that 😉 Just sharing as this might help with our solution. There maybe also be parts of code, we need to consider for our solution. |
@swissspidy morning! While testing this PR (#11732) I noticed that Saturday, Sunday and Monday can be set as 1st days, but Friday can't. Is this something that we need to fix in this PR or can we create a new one for this scenario? |
Nothing we can fix. The calendar we use does not support it. |
Thanks! So the PR is approved. |
Bug Description
WordPress has a setting for selecting the first day of the week. However, our date picker does not respect that.
See related WP core issue. #55826.
Expected Behaviour
Steps to Reproduce
Screenshots
Additional Context
The text was updated successfully, but these errors were encountered: