Make un-hover delay configurable

aaron.ferris → made their first commit to this issue’s fork.

Made an initial MR for this - we could always add an update hook to make sure this setting exists, ive coded defensively for it.

This is awesome @aaron.ferris, thanks!

It works perfectly, and bumping "Hover Intent Timeout (ms)" up to 500 ms or even 750 ms allows for non-precise mousing, without having to start all over with a multi-level traversal, due to slipping out of the menus accidentally. I'll set to RTBC, and if more code is on the way, it can always be reverted to "Needs work".

PS. Setting timeout to 0 ms and speed-running Drupal menus, traversing to the deepest level of all menus could be a new sports discipline :-)

No problem, glad it’s working well for you.

I can confirm that this does indeed work well. In addition, by moving the hoverintent to a drupal behavior instead of $(document).ready it fixes another issue I've been seeing on some of my sites where hoverintent failed to work at all.

Thanks a lot everyone for the great work on this issue, it's greatly appreciated!

I've rebased/fixed a few conflicts that were blocking MR!99 and it should be good now.

I haven't tested the feature yet, but after a very quick review of the MR, it looks good overall, great job! 🙂

I think there are a few more changes I'd like to add to the MR, in particular, I think the whole hoverintent feature should probably be moved out from the admin_toolbar_tools module to the admin_toolbar.
This feature "should" work without the admin_toolbar_tools module, so the config variable hoverintent_functionality should be move out to admin_toolbar, impacting the schema, the settings forms and probably the install files of the module (to update module's configuration on all sites).
I'll then add a form state to the added hover_intent_timeout config field to hide or show if the functionality is enabled or disabled.

Lastly, we'll mostly likely need to add Functional tests for this:
https://git.drupalcode.org/project/admin_toolbar/-/blob/3.x/tests/src/Fu...

More work is needed on this, so I'm setting this to Needs work and will circle back on this in the next few days: I'm going to need a few hours to get this properly fixed.

Bringing this one back up to the top of the list.
The good news is that the changes at this point seem pretty straight forward, therefore there are great chances this feature could be merged in the coming days/weeks.

At which point the MR will be set back to Needs review and we will need you help testing and reviewing again.
Thanks again!

We'll see, but I'll most likely split this into a separate ticket to get the hoverintent_functionality moved out from admin_toolbar_tools to admin_toolbar and come back to this issue right after to add the state on the field added by the MR.

More coming in the next few days.
Cheers!