[1.0.0-alpha1] Contextual menu looks broken

Created on 21 June 2025, 4 months ago

Problem/Motivation

The contextual menu that shows the <sl-menu> is currently not really visible. When removing the type="checkbox" attribute from them we can at least read their names.

Steps to reproduce

  1. Open Layers
  2. Right-click on an element

Proposed resolution

Either we remove type="checkbox" here or there is another solution to restore the planned design for this.

🐛 Bug report
Status

Active

Version

1.0

Component

display_builder_ui

Created by

🇩🇪Germany yannickoo Berlin

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @yannickoo
  • 🇫🇷France pdureau Paris

    Hi Yannick,

    Thanks for your issue. I am not reproducing it exactly.

    With Olivero theme, when Layers panel is in the sidebar, instead of having a broken sl-menu, I have the browser menu:

    With UI Suite Bootstrap theme, the right-click sl-menu is always OK:

    So, Yannickoo, can you tell us more about your current setup?

  • 🇫🇷France mogtofu33

    What @yannickoo is pointing is that the menu can not be inspected (right click).
    The type="checkbox" is used to make the click event work and propagate as seen in the shoelace menu.
    I don't think we can remove this type, and I am not sure of the level of the issue here. Is there an accessibility problem?

  • 🇩🇪Germany yannickoo Berlin

    I agree that removing the type="checkbox" needs to be there but I am still wondering what could be the reason that the contextual menu doesn't render properly in a custom theme.

  • 🇫🇷France mogtofu33

    doesn't render properly in a custom theme.

    Could you be more specific? Then the issue could be a conflict on the css with the custom theme?

  • 🇫🇷France pdureau Paris

    Waiting for answer. Moved to alpha4.

  • 🇩🇪Germany yannickoo Berlin

    After a while I was able to figure out the root cause for this: Ensure that your normalize / reset CSS code is not having generic selectors like [type="checkbox"] but input[type="checkbox"] instead to avoid overriding the CSS style of the contextual menu items 💡

    Closing this one but might be helpful for future ✨

Production build 0.71.5 2024