-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prevent duplicate highlights in mega menu #6016
Conversation
8039b25
to
42cdc26
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me 👍 . Had a question about the test.
}, | ||
]) | ||
|
||
def test_conversion_output_selection_ignores_query_string(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are your thoughts about adding a test for the specific behavior that's added in this PR? I might be wrong, but this test seems more like #6012
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, you're totally right. I'll add one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@schbetsy I've added a test for this behavior; would you mind re-reviewing when you get a chance?
26cfac3
to
cc0f1c7
Compare
cc0f1c7
to
ce76a22
Compare
This change modifies the existing mega menu logic to prevent duplicate highlights in the mega menu, for cases where a menu link is duplicated between submenus. The code now ensures that only at most one submenu and at most one link gets highlighted.
This change modifies the existing mega menu logic to prevent duplicate highlights in the mega menu, for cases where a menu link is duplicated between submenus.
The code now ensures that only at most one submenu and at most one link gets highlighted.
How to test this PR
Edit the mega menu to duplicate a link between two submenus. Internally, see UCP PR deployment for this running live with an example.
Checklist