Skip to content

Conversation

@lauraneto
Copy link
Contributor

@lauraneto lauraneto commented Oct 1, 2025

Description

This PR improves the user experience when dealing with missing property editors and property editor UIs in two key areas:

Data Type Workspace Improvements

  • Enhanced visibility: Missing property editors and UIs now display clearly instead of showing nothing/blank selection
  • Visual indicators: Missing editors are highlighted with red styling to make them more noticeable
  • Save validation: Added validation that prevents saving data types with missing property editors/UIs and displays helpful error messages
image image

Property Editor UI Feedback

  • Clear feedback: Properties missing a property editor UI now display with clear, actionable feedback
  • Developer guidance: Added helpful guidance on how to resolve missing property editor UI issues
  • Improved messaging: Enhanced error messages to be more specific and actionable for developers implementing custom property editor UIs
image image

Copy link
Member

@nielslyngsoe nielslyngsoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really great — Thanks a lot!

@lauraneto lauraneto marked this pull request as ready for review October 2, 2025 12:08
Copilot AI review requested due to automatic review settings October 2, 2025 12:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves the user experience when dealing with missing property editors and property editor UIs in Umbraco CMS by providing clearer feedback and validation. The changes include enhanced visibility of missing editors with visual indicators, improved error messages, and prevention of saving data types with missing configurations.

  • Refactored missing property editor components to share common functionality through a base class
  • Added validation to prevent saving data types with missing property editors/UIs
  • Enhanced error messages and visual feedback for missing property editor UIs

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
property-editor-ui-missing.element.ts Refactored to extend base class and simplified implementation
property-editor-ui-missing-ui.element.ts New component for handling missing property editor UI scenarios
property-editor-ui-missing-base.element.ts New base class containing shared functionality for missing property editor components
manifests.ts Added manifest for new missing UI component and updated existing manifest
data-type-details-workspace-view.element.ts Simplified to use new property editor picker component
data-type-details-workspace-property-editor-picker.element.ts New component with validation for missing property editors/UIs
data-type-workspace.context.ts Added fallback logic for property editor schema alias
property.element.ts Added fallback to missing UI component when property editor UI not found
constants.ts Added constants for missing property editor UI aliases
validation.controller.ts Enhanced error message for missing validation messages
lang files Added localization strings for new error messages

@lauraneto lauraneto marked this pull request as draft October 2, 2025 13:29
@lauraneto
Copy link
Contributor Author

Converted back to draft as this caused a failing smoke test: can change property editor in a data type.

@lauraneto lauraneto marked this pull request as ready for review October 2, 2025 14:57
@lauraneto lauraneto merged commit e6d1d7a into main Oct 2, 2025
26 of 27 checks passed
@lauraneto lauraneto deleted the v16/bugfix/data-type-workspace-missing-editor branch October 2, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants