-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Property Editors: New Date Time property editors #19915
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
Property Editors: New Date Time property editors #19915
Conversation
…. Display UTC offset instead of short offset name in label.
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.
Pull Request Overview
Copilot reviewed 83 out of 84 changed files in this pull request and generated 4 comments.
Files not reviewed (1)
- src/Umbraco.Web.UI.Client/package-lock.json: Language not supported
...s.UnitTests/Umbraco.Infrastructure/PropertyEditors/TimeOnlyPropertyIndexValueFactoryTests.cs
Outdated
Show resolved
Hide resolved
...Umbraco.Infrastructure/PropertyEditors/DateTimeWithTimeZonePropertyIndexValueFactoryTests.cs
Show resolved
Hide resolved
.../Umbraco.Infrastructure/PropertyEditors/DateTimeUnspecifiedPropertyIndexValueFactoryTests.cs
Show resolved
Hide resolved
...s.UnitTests/Umbraco.Infrastructure/PropertyEditors/DateOnlyPropertyIndexValueFactoryTests.cs
Show resolved
Hide resolved
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.
This is so good @lauraneto - really great piece of work.
I had no further comments - just pushed up a few additional XML header comments to complete the documentation on the new classes you've introduced.
Did some testing of all different types, and looks to work flawlessly to me:


And from delivery API output:
"dateOnly": "2025-09-23",
"timeOnly": "12:14:06",
"dateUnspecified": "2025-09-23T12:14:00",
"dateLocal": "2025-09-23T12:14:00+02:00",
"dateUtc": "2025-09-23T12:14:00+00:00",
"dateWithTimezones": "2025-09-23T12:14:00+10:00"
...mbraco.Web.UI.Client/src/packages/core/components/input-time-zone/input-time-zone.element.ts
Outdated
Show resolved
Hide resolved
...rc/packages/property-editors/time-zone-picker/property-editor-ui-time-zone-picker.element.ts
Outdated
Show resolved
Hide resolved
…operty-editor # Conflicts: # src/Umbraco.Web.UI.Client/src/packages/core/validation/mixins/form-control.mixin.ts
…operty-editor # Conflicts: # src/Umbraco.Web.UI.Client/package-lock.json
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.
Looks good
* Upgrade to Tiptap v3 * Uses `@ts-expect-error` to ignore the TS complication errors These can be removed once Tiptap has resolved the TypeScript definitions. * Off-topic: corrected `flags` property in the mock data Added in PR #19915 * Update src/Umbraco.Web.UI.Client/src/packages/tiptap/extensions/link/link.tiptap-extension.ts Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Jacob Overgaard <[email protected]>
The documentation on the new property editors can be found in: https://docs.umbraco.com/umbraco-cms/17.latest/fundamentals/backoffice/property-editors/built-in-umbraco-property-editors/date-time-editor