-
Notifications
You must be signed in to change notification settings - Fork 0
Datefield enhancements. #337
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
Conversation
|
||
The `DateField` component stores its value internally as a `LocalDate` object, representing a date without time or timezone information. This allows for accurate handling of calendar-based input across different systems. | ||
|
||
While the **visual display adapts to the user's browser locale**, ensuring regionally familiar formatting (e.g., MM/DD/YYYY in the USA or DD.MM.YYYY in Europe), the internal parsing and programmatic interaction always rely on the fixed format: **`yyyy-MM-dd`**. |
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.
📝 [vale] reported by reviewdog 🐶
[Google.Acronyms] Spell out 'YYYY', if it's unfamiliar to the audience.
|
||
The `DateField` component stores its value internally as a `LocalDate` object, representing a date without time or timezone information. This allows for accurate handling of calendar-based input across different systems. | ||
|
||
While the **visual display adapts to the user's browser locale**, ensuring regionally familiar formatting (e.g., MM/DD/YYYY in the USA or DD.MM.YYYY in Europe), the internal parsing and programmatic interaction always rely on the fixed format: **`yyyy-MM-dd`**. |
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.
📝 [vale] reported by reviewdog 🐶
[Google.Acronyms] Spell out 'USA', if it's unfamiliar to the audience.
|
||
The `DateField` component stores its value internally as a `LocalDate` object, representing a date without time or timezone information. This allows for accurate handling of calendar-based input across different systems. | ||
|
||
While the **visual display adapts to the user's browser locale**, ensuring regionally familiar formatting (e.g., MM/DD/YYYY in the USA or DD.MM.YYYY in Europe), the internal parsing and programmatic interaction always rely on the fixed format: **`yyyy-MM-dd`**. |
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.
📝 [vale] reported by reviewdog 🐶
[Google.Acronyms] Spell out 'YYYY', if it's unfamiliar to the audience.
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.
In the section headings, when using methods, let's either always use the parenthesis (setText()) or not (setMin). I'd actually change the last two headings to "Setting the min and max" and combine that section into one.
/rebase |
693356d
to
987ae30
Compare
- Removed "Localized display" section (last bullet point in Issue 58) - Created subsection titles for "Min and max value" - Put all date formats into code styling - Added "Updated" tag to the sidebar - Changed tip about `IllegalArgumentException` to a warning
987ae30
to
8c6998b
Compare
#57
#58