A few improvement proposals.

Add form:

  • Redirect to /admin/structure/display-builder after entity creation instead of showing a new create form
  • Default values: the same as display_builder.display_builder.default

Add & edit forms:

  • And add a checkbox to activate the related permission by role (like what Text Formats are doing)?
  • Remove shoelace library from display builder config entity because it may not be necessary to have different config by Display Builder and because it breaks the previews in UI Patterns Library.

Collection page (/admin/structure/display-builder)

  • Add a col with roles having the related permission in /admin/structure/display-builder (like /admin/config/content/formats is doing)
  • Remove machine name col
  • Reorder by adding a weight property and implementing DraggableListBuilder. Add a little explanation, like Text Format. (and update ConfigFormBuilder::buildDisplayBuilder())
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

pdureau created an issue. See original summary.

pdureau’s picture

Assigned: Unassigned » pdureau
Issue summary: View changes
Status: Active » Needs work
Issue tags: -display_builder-1.0.0-beta +display_builder-1.0.0-alpha3

A few improvement proposals.

Add form:

  • Redirect to /admin/structure/display-builder after entity creation instead of showing a new create form >> we keep it like that to allow island configuration
  • Default values: the same as display_builder.display_builder.default

Add & edit forms:

  • ✅ And add a checkbox to activate the related permission by role (like what Text Formats are doing)?
  • Remove shoelace library from display builder config entity because it may not be necessary to have different config by Display Builder and because it breaks the previews in UI Patterns Library.

Collection page (/admin/structure/display-builder)

  • ✅ Add a col with roles having the related permission in /admin/structure/display-builder (like /admin/config/content/formats is doing)
  • ✅ Remove machine name col
  • ✅ Reorder by adding a weight property and implementing DraggableListBuilder. Add a little explanation, like Text Format. (and update ConfigFormBuilder::buildDisplayBuilder())
pdureau’s picture

Issue summary: View changes
pdureau’s picture

pdureau’s picture

Remove shoelace library from display builder config entity because it may not be necessary to have different config by Display Builder and because it breaks the previews in UI Patterns Library.

Moved to its own issue: #3538207: Remove shoelace library from display builder config

pdureau’s picture

Assigned: pdureau » mogtofu33
Status: Needs work » Needs review

  • mogtofu33 committed f0827701 on 1.0.x authored by pdureau
    Issue #3537137 by pdureau: Display Builder config entity UX proposals
    
mogtofu33’s picture

Assigned: mogtofu33 » Unassigned
Status: Needs review » Fixed

  • mogtofu33 committed 2adfd4fa on 1.0.x
    issue #3537137 by pdureau, mogtofu33: UX enhancement, use 'profile' term...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.