WordPress.org

Make WordPress Core

#23752 closed enhancement (fixed)

Update hoverIntent to r7

Reported by: joel_birch Owned by: ocean90
Milestone: 3.6 Priority: normal
Severity: normal Version:
Component: External Libraries Keywords: has-patch
Focuses: Cc:

Description

I'd like to request that hoverIntent be updated to hoverIntent r7.

hoverIntent.js has just been updated to support event delegation whilst still being backward-compatible. My Superfish menu plugin requires this, meaning that when users set hoverIntent as a dependency they have to dequeue WordPress's version of hoverIntent and requeue the updated version, which is awkward, and their dropdown menus will not work at all without it.

Also, whatever WordPress uses hoverIntent for should benefit from using event delegation in future.

External library: https://github.com/briancherne/jquery-hoverIntent

Attachments (3)

hoverintentr7.diff (11.7 KB) - added by mjbanks 16 months ago.
upgrade hoverintent to r7, both full and minified version (via uglify)
hoverintent-r7-v2.diff (9.4 KB) - added by mjbanks 16 months ago.
upgrade hoverintent to r7, full version only
admin-bar-js.diff (2.6 KB) - added by mjbanks 14 months ago.

Download all attachments as: .zip

Change History (12)

comment:1 SergeyBiryukov16 months ago

  • Milestone changed from Awaiting Review to 3.6

mjbanks16 months ago

upgrade hoverintent to r7, both full and minified version (via uglify)

comment:2 mjbanks16 months ago

Added patch to update hoverintent to r7. This new version required jQuery 1.9.1+, which is what's being bundled with 3.6, so it runs with no problem (as long as people aren't using their own version of jQuery, which they shouldn't be anyway).

comment:3 follow-up: mjbanks16 months ago

  • Keywords has-patch added

Wasn't sure if I should add has-patch here or a Core maintainer does that (my first Core patch, be gentle).

comment:4 markoheijnen16 months ago

Minified version is not needed in the patch.

mjbanks16 months ago

upgrade hoverintent to r7, full version only

comment:5 mjbanks16 months ago

Added revised patch with only the full version

comment:6 in reply to: ↑ 3 rmccue16 months ago

Replying to mjbanks:

Wasn't sure if I should add has-patch here or a Core maintainer does that (my first Core patch, be gentle).

If you add a patch, you should add the has-patch keyword yourself. Most committers look at tickets with has-patch first so they can commit those. If it needs extra testing, add needs-testing, and if it needs someone else to look over it, add dev-feedback. Simple! :)

comment:7 follow-up: ocean9014 months ago

We should remove the minfied version at some time, see #21769.

@mjbanks:

Can you please update wp-includes/admin-bar,js too? It includes the minified version.

mjbanks14 months ago

comment:8 in reply to: ↑ 7 mjbanks14 months ago

Replying to ocean90:

We should remove the minfied version at some time, see #21769.

@mjbanks:

Can you please update wp-includes/admin-bar,js too? It includes the minified version.

Sure thing! Updated and diff attached.

comment:9 ocean9014 months ago

  • Owner set to ocean90
  • Resolution set to fixed
  • Status changed from new to closed

In 24259:

hoverIntent r7. props mjbanks. fixes #23752.

Note: See TracTickets for help on using tickets.