Closed (fixed)
Project:
hoverIntent
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 Oct 2011 at 22:09 UTC
Updated:
18 Feb 2017 at 16:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sanduhrsAttached is a patch that upgrades the module to D7.
It also upgrades to the latest hoverIntent script.
As there's no D7 version of JQ it's just hook_library().
Comment #2
DruBerrymore commentedCould somebody please apply the patch in #1, and then make a 7.x-1.x-dev release.
Could somebody also be a co-maintainer.
This module is needed by the Giga Menu module.
Comment #3
a_c_m commentedsanduhrs - happy to give you co-maintainership if you want it. (Or would if i could work out how :P) May need help from aaron as i'm not 100% i have the power.
Comment #4
DruBerrymore commentedWould be happy to test it.
Comment #5
sanduhrsShure. On the project page click on the Maintainers tab - if you can ;)
[1] http://drupal.org/node/213818/maintainers
Comment #6
a_c_m commentedYeah, i don't have access, sorry. Might need to message aaron direct. I will see if hes on IRC.
Comment #7
aaron commentedFixed the access, a_c_m. Thanks for your help, sanduhrs, you're in now!
Comment #8
aaron commentedoops, status
Comment #9
sanduhrsThanks, aaron.
Just added a d7 dev release.
Please test and report back.
Comment #10
DruBerrymore commentedDon't see the d7 dev release on the project page.
Comment #11
sanduhrsWell, it takes some time for the release to show up.
But you can already clone in from the repository: http://drupal.org/node/213818/git-instructions/7.x-1.x
Comment #12
stevetweeddale commentedJust uploading this here in case the maintainers want it (I'll be using the patch in a drush make file regardless): I've added a dependency on libraries module and removed the hoverIntent js file from the module itself - the module would now just exposed the library to hook library. I believe this is the standard way of doing things nowadays. Hopefully at some point in the future libraries will be clever enough to not even require this kind of 'wrapper' module.
Comment #13
boonthanom commentedHi,all
i would like to know step by step to run patch , so that my site have error messages as below
Hover intent not "installed in sites/default/libraries" The module will not work without it.
Hover intent not "installed in sites/default/libraries" The module will not work without it.
Thank you
Comment #14
stevetweeddale commentedHi Boonthanom,
There is some javascript (the 'hoverintent' library itself) that was included in the module, but my patch removes it. This is closer to the general standard for this kind of module, where typically you download the integration module from Drupal.org and the external library from wherever the author publishes it.
So if you want to use my patched version, you'll need to download the library manually and put in sites/all/libraries. Otherwise, don't bother with my patch!
Comment #15
stephen ollmanSteve, what is the name of the folder that should be created in the libraries folder?
Is it 'hoverintent'?
I'm trying to use this within the muchmenus js code, replacing:
$('.muchomenu-parent-title').bind('mouseover', muchomenu_open);
$('.muchomenu-parent-title').bind('mouseout', muchomenu_timer);
with
$('.muchomenu-parent-title').hoverIntent(muchomenu_open, muchomenu_timer);
However after applying the patch and creating the library folder and file and also updating my template.php file, I'm still getting the error: TypeError: $(...).hoverIntent is not a function
Comment #17
oadaeh commentedI created a 7.x-2.x branch and committed different, minimal changes there to allow this module to work with the 2.x branch of the Libraries API module.
As there are a number of existing users for the 7.x-1.x branch, I am not making any changes in that branch that might break their sites.
Feel free to upgrade your hoverIntent and Libraries API modules to their 2.x branches to take advantage of the updates.