Skip to content
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

Consider support for multiple levels of submenu #8

Open
mrwweb opened this issue May 31, 2021 · 4 comments
Open

Consider support for multiple levels of submenu #8

mrwweb opened this issue May 31, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@mrwweb
Copy link
Owner

mrwweb commented May 31, 2021

While I don't like it, it would make the script more useful to more people if this supported multiple levels of submenus. That said, doing then must not come at the expense of being able to make "mega menus" that show nested lists within the top-level submenu. Therefore, the secondary submenu feature would need to be configurable in a way that makes it opt-in.

@timburg1
Copy link

I'd really appreciate support for multi-level menus. The accessibility of this offering is great - I'm a blind developer supporting blind end-users, so this is vital for me. However, I have multiple projects that need to create multi-level menus dynamically by reading in an XML file that defines the interface and this is where Clicky falls short.

@ajplopez
Copy link

ajplopez commented Feb 2, 2024

I third this (no pun intended). This would be a fantastic addition and I would use this script even more.

@mrwweb
Copy link
Owner Author

mrwweb commented Feb 12, 2024

@ajplopez I'd love to make it easier to handle that, although there is one thing you can do to make that easier. Take a look at the "Custom Submenu Selector":

<ul class="clicky-menu" data-clicky-submenu-selector=".clicky-menu > li > ul">
    <!-- menu items -->
</ul>```

Doing that will only target the first level of nested menus as hidable submenus. You'll still need to do a bit of work to reset some of the sites (`visibility: visible` for sure), but that at least gets you past the script trying to show/hide nested/tertiary submenus.

@ajplopez
Copy link

@mrwweb Yes! Sorry, I realized after what I needed so I deleted that last comment. Got it working great.

@mrwweb mrwweb added the enhancement New feature or request label Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants