Skip to content

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

Merged
merged 5 commits into from
Jun 25, 2025
Merged

Datefield enhancements. #337

merged 5 commits into from
Jun 25, 2025

Conversation

EHandtkeBasis
Copy link
Member


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`**.

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`**.

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`**.

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.

Copy link
Member

@MatthewHawkins MatthewHawkins left a 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.

MatthewHawkins
MatthewHawkins previously approved these changes May 27, 2025
@bbrennanbasis
Copy link
Member

/rebase

@github-actions github-actions bot force-pushed the datefield-enhancement branch from 693356d to 987ae30 Compare June 17, 2025 15:43
@github-actions github-actions bot requested a review from bbrennanbasis as a code owner June 17, 2025 15:43
EHandtkeBasis and others added 5 commits June 23, 2025 10:34
- 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
@hyyan hyyan merged commit 7996dbd into main Jun 25, 2025
2 checks passed
@hyyan hyyan deleted the datefield-enhancement branch June 25, 2025 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding Field Value Details to DateField Documentation Improve DateField Validation Labels and Behavior
4 participants