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

[Feature]: Timings of Navigation Menu are not customizable #1578

Open
2 tasks
spaceemotion opened this issue Jan 24, 2025 · 2 comments
Open
2 tasks

[Feature]: Timings of Navigation Menu are not customizable #1578

spaceemotion opened this issue Jan 24, 2025 · 2 comments
Labels

Comments

@spaceemotion
Copy link

Describe the feature

We wanted to use the NavigationMenu for our website, but encountered a couple issues:

The menu root has config options for the hover delays and whatnot, but the triggers seem to have a hard-coded timeout of 300ms. This makes it impossible to quickly fly over the menu options using the mouse, as this value prevents menus from being reopened right away.

https://github.com/unovue/radix-vue/blob/8a530ba61624036738c06d08940571485743fd2c/packages/core/src/NavigationMenu/NavigationMenuTrigger.vue#L40

We'd love to have no timers whatsoever, so as soon as the mouse enters a trigger, the menu opens, and when another trigger gets activated, the menu switches right away.

You can see the desired "speed" by going to the GitHub page when logged out (or in private mode), and flicking the cursor over the navigation bar at the top:

Image

I also noticed that there's no way to turn any of the event listeners off when passing a value of 0 to the delay props.

Additional information

  • I intend to submit a PR for this feature.
  • I have already implemented and/or tested this feature.
@zernonia
Copy link
Member

zernonia commented Feb 3, 2025

This is a good feature to have. Will include this in v2 (as we are freezing the feature on v1 🙏🏼) 😁

@zernonia zernonia added the v2 label Feb 3, 2025
@zernonia zernonia added this to v2 Feb 3, 2025
@zernonia zernonia moved this to Backlog in v2 Feb 3, 2025
@spaceemotion
Copy link
Author

@zernonia just wanted to let you know that we decided to build the navigation ourselves since we couldn't fix all the bugs. We're not using radix-vue at all right now, but would definitely love to switch to a more ARIA-compliant solution when v2 is out :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Backlog
Development

No branches or pull requests

2 participants