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
weightproperty and implementingDraggableListBuilder. Add a little explanation, like Text Format. (and updateConfigFormBuilder::buildDisplayBuilder())
Issue fork display_builder-3537137
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
Comment #2
pdureau commentedA 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 configurationdisplay_builder.display_builder.defaultAdd & edit forms:
Collection page (/admin/structure/display-builder)
weightproperty and implementingDraggableListBuilder. Add a little explanation, like Text Format. (and updateConfigFormBuilder::buildDisplayBuilder())Comment #3
pdureau commentedComment #4
pdureau commentedComment #5
pdureau commentedMoved to its own issue: #3538207: Remove shoelace library from display builder config
Comment #7
pdureau commentedComment #9
mogtofu33 commented