- Issue created by @finnsky
- ๐ช๐ธSpain ckrina Barcelona
Good point. Basically because we might end up having 2 custom menus with the same URLS. And we need to set a pattern.
I'd say the easier way should be the menu closer on the top should have more priority: it'd be faster to reach and to visually locate. An alternative would be to add weight to each menu, but imho that would add unnecessary complexity.
I know this goes against removing the active state from Shortcuts that we worked on, but we'll work on changing a few things for that menu: we'll add the links directly as items in the menu.
- ๐ฎ๐ณIndia prashant.c Dharamshala
Prashant.c โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia prashant.c Dharamshala
One other scenario or bug I found related to multiple active links
such as when a URL strings matches will multiple anchor links in the menu multiple links getting active for example in user menu
href
forView Profile
is/user
andhref
forEdit Profile
is/user/1/edit
. In this case both links highlighted.I have tried to fix it by making some tweaks, there might be other scenarios as well.
Thanks!
- Merge request !154Fixed multiple menu active having current URL substrings โ (Open) created by prashant.c
- Status changed to Needs work
12 months ago 1:23pm 22 December 2023 - ๐ฎ๐ณIndia prashant.c Dharamshala
Changing to NW as main issue regarding 2 menu active links still needs work as mentioned in #2 โจ Decide how to control 2 active urls Active
- ๐ฎ๐ณIndia prashant.c Dharamshala
In my opinion, it's acceptable for multiple menu items to share the same link and display them as active. This is because, if we prioritize these links or don't highlight others as active, users and administrators may not be aware that there are multiple links with the same URL. Additionally, there could be intentional reasons for having multiple links with the same URL. As a result, this should not pose any harm, but further discussion or feedback is encouraged.
Thank you.
- Status changed to Closed: works as designed
9 months ago 2:54pm 5 April 2024 - ๐บ๐ธUnited States KeyboardCowboy Denver, CO, USA
We're closing this issue as it works as designed. With the new Drawer designs this is no longer a UX issue. We can open a new issue if we find this to be a problem in future iterations.