WTH week starts on Sunday?

chocomega · October 3, 2022, 2:57pm

DateTime picker first day of week should follow locale

opened 09:29AM - 19 Dec 21 UTC

Bug

### Checklist - [X] I have updated to the latest available Home Assistant versi…on. - [X] I have cleared the cache of my browser. - [X] I have tried a different browser to see if it is related to my browser. ### Describe the issue you are experiencing The datetime picker has Sunday as the first day of week ### Describe the behavior you expected The datetime picker should follow my locale or, if not possible, fallback to ISO 8601 value of Monday ### Steps to reproduce the issue 1. Set my language to French 2. Go to logbook and open datetime picker 3. See that first day of week is Sunday ... ### What version of Home Assistant Core has the issue? 2021.12.3 ### What was the last working version of Home Assistant Core? _No response_ ### In which browser are you experiencing the issue with? _No response_ ### Which operating system are you using to run this browser? _No response_ ### State of relevant entities _No response_ ### Problem-relevant frontend configuration _No response_ ### Javascript errors shown in your browser console/inspector _No response_ ### Additional information _No response_

chocomega · October 5, 2022, 2:57pm

It looks like it’s being worked on

github.com/home-assistant/frontend

Add first weekday option in profile

home-assistant:dev ← home-assistant:first_weekday_profile

opened 04:17PM - 04 Oct 22 UTC

piitaya

+253 -14

## Proposed change This PR allow user to select the first day of the week m…anually or use language option. The language option uses `Int.Locale.weekInfo` but it's not always possible to determine the first day of week because it's not supported in all browsers and no polyfills exist. https://caniuse.com/mdn-javascript_builtins_intl_locale_weekinfo So, a fallback has been added to https://github.com/gamtiq/weekstart for browsers that don't support `Int.Locale.weekInfo`. ![Capture d’écran 2022-10-05 à 11 34 06](https://user-images.githubusercontent.com/5878303/194030271-a61f8435-d1f7-4a93-9302-8c6a7e6dc32f.png) Fixed time components: - [x] date input - [x] date range picker - [x] schedule - [x] calendar - [x] automation time condition (re-order days) ## Type of change  - [ ] Dependency upgrade - [ ] Bugfix (non-breaking change which fixes an issue) - [x] New feature (thank you!) - [ ] Breaking change (fix/feature causing existing functionality to break) - [ ] Code quality improvements to existing code or addition of tests ## Example configuration  ```yaml ``` ## Additional information  - This PR fixes or closes issue: fixes #10970, fixes #12635 - This PR is related to issue or discussion: - Link to documentation pull request: ## Checklist  - [ ] The code change is tested and works locally. - [ ] There is no commented out code in this PR. - [ ] Tests have been added to verify that the new code works. If user exposed functionality or configuration variables are added/changed: - [ ] Documentation added/updated for [www.home-assistant.io][docs-repository]  [docs-repository]: https://github.com/home-assistant/home-assistant.io

frenck · October 26, 2022, 4:50pm

It has been addressed in the above-linked PR and will be part of the 2022.11 release. I’ve marked the above answer as the solution.