- Issue created by @lostcarpark
- 🇺🇸United States jayhuskins
That label comes from the core menu block configuration. I kept in place mostly so that site builders already familiar with that setting wouldn't be confused, but there's nothing to stop us from relabelling it and/or adding better help text.
- 🇺🇸United States jayhuskins
As mention by @ressa in this issue (#3401517), it may be better to have the "Enable all menu links" option enabled by default. In that case we could hide the "Enable all menu links" option and add an "Only include expanded submenus" option that's disabled by default.
- 🇮🇪Ireland lostcarpark
That label comes from the core menu block configuration.
I think that's why it was confusing to me. I see the logic of using consistent termonology with the core block, but where functionality differs
In that case we could hide the "Enable all menu links" option and add an "Only include expanded submenus" option that's disabled by default.
Do you mean "remove" rather than "hide" the "Enable all menu links"? I think this would make sense.
- 🇩🇰Denmark ressa Copenhagen
Do you mean "Expand all menu links" @jayhuskins? Or is there an "Enable all menu links" option somewhere?
But yes, I think the "Expand all menu links" option should be enabled by default :)
- 🇺🇸United States jayhuskins
Do you mean "Expand all menu links" @jayhuskins? Or is there an "Enable all menu links" option somewhere?
@ressa, Whoops, yes I mean "Expand all menu links" :)
- 🇩🇰Denmark ressa Copenhagen
Great @jayhuskins :) I am having difficulties visualizing '... hide the "Enable Expand all menu links" option and add an "Only include expanded submenus" option that's disabled by default.' ...
- 🇩🇰Denmark ressa Copenhagen
So this looks like it stalled ... though it seems like you both agree on the solution. I am not certain what it will look like, but as long as the result will be an "Expand all menu links" situation, feel free to go ahead :)
- 🇺🇸United States jayhuskins
Here's where I'm at now:
- "Expand all menu links" is a confusing option in the disclosure menu context
- But the setting needs to be supported in some fashion
- By default, all menu link items should be rendered (even if they haven't been set to "expanded")
- So "Expand all menu links" should be true by default
- We can relabel the option to something like "Build all submenus"
- Or we can invert the option (disabled by default) and label it something like "Only build enabled submenus"
- I like the terms "build" and "render" because the setting controls what markup gets added, not necessarily what gets displayed
For reference this is the setting on the menu link item that we are overriding: