[PR #971] DatePicker: handle AM/PM consistently #2439

Open
opened 2026-01-29 18:18:50 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/radzenhq/radzen-blazor/pull/971

State: closed
Merged: Yes


  • simplify the 'ToggleAmPm' function,
  • handle an AM/PM formatted hour input consistently (fixes #939) – note that it also prevents AM/PM from changing when you change the hour to 12,
  • fix possible out-of-range values of hour/minutes/seconds on input*

*However I couldn't reproduce #109 after disabling oninput events anyway… so perhaps the private variables hour/minutes/seconds along with related on-input functions are no longer needed?

**Original Pull Request:** https://github.com/radzenhq/radzen-blazor/pull/971 **State:** closed **Merged:** Yes --- - simplify the 'ToggleAmPm' function, - handle an AM/PM formatted hour input consistently (fixes #939) – note that it also prevents AM/PM from changing when you change the hour to 12, - fix possible out-of-range values of hour/minutes/seconds on input* *However I couldn't reproduce #109 after disabling oninput events anyway… so perhaps the private variables _hour/minutes/seconds_ along with related on-input functions are no longer needed?
claunia added the pull-request label 2026-01-29 18:18:50 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/radzen-blazor#2439