Problem/Motivation

UI Suite Daisy UI has this account menu--account.html.twig:

{% for item in items %}
  {{ include('ui_suite_daisyui:button', {
  url: item.url.toString(),
  label: item.title,
  }, with_context: false) }}
{% endfor %}

We can drag each button one by one and that must not be possible.

Proposed resolution

Like what we already do with empty elements, wrap those blocks before rendering them in BuilderPanel.

Issue fork display_builder-3557112

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:

About issue forks

Comments

pdureau created an issue. See original summary.

pdureau opened merge request !160

pdureau’s picture

pdureau commented
Assigned: pdureau » mogtofu33
Status: Active » Needs review

mogtofu33 made their first commit to this issue’s fork.

mogtofu33’s picture

mogtofu33 commented
Related issues: +#3557112: BuilderPanel: blocks with many root HTML elements are not draggables

Added a follow up #3557196: Add tests for BuilderPanel::buildSingleBlock() as the buildSingleBlock method is complex and need some tests.

mogtofu33’s picture

mogtofu33 commented
Assigned: mogtofu33 » Unassigned
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • mogtofu33 committed fc7a2a40 on 1.0.x authored by pdureau 
    fix: #3557112 BuilderPanel: blocks with many root HTML elements are not...

Status: Fixed » Closed (fixed)

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