Closed HenryLoenwind closed 2 months ago
Describe the bug. In develop, StoryMenu is broken.
I tracked it down to
251: jQuery(document).on(':uiupdate${EVENT_NS}', () => {
This installs an event handler for:
Not quite what was intended, as ${EVENT_NS} should be expanded to .ui-bar.
${EVENT_NS}
.ui-bar
Replacing the ' with backticks fixes that, but exposes a second issue:
The new code adds an extra <ul>:
<ul>
A possible fix is:
#menu ul ul { display: contents; }
But naturally, getting rid of the extra <ul>s would be the cleaner approach.
Thanks.
Describe the bug. In develop, StoryMenu is broken.
I tracked it down to
This installs an event handler for:
Not quite what was intended, as
${EVENT_NS}
should be expanded to.ui-bar
.Replacing the ' with backticks fixes that, but exposes a second issue:
The new code adds an extra
<ul>
:A possible fix is:
But naturally, getting rid of the extra
<ul>
s would be the cleaner approach.