Skip to content

[pull] main from jquery:main#66

Merged
pull[bot] merged 1 commit into
TheRakeshPurohit:mainfrom
jquery:main
Mar 21, 2025
Merged

[pull] main from jquery:main#66
pull[bot] merged 1 commit into
TheRakeshPurohit:mainfrom
jquery:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Mar 21, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

As of gh-2338, if one has loaded the jQuery MouseWheel plugin, the `mousewheel`
handler would fire the `wheel` one, but the `wheel` one would also run in
response to the native `wheel` event, resulting in double the distance handled
by the spinner. To prevent the issue, only fire the `wheel` handler from inside
the `mousewheel` on if the event was triggered by jQuery - jQuery will not care
that the underlying event is `wheel` and will only fire handlers for
`mousewheel`.

Also, add an iframe test using jQuery MouseWheel to not affect all the other
tests.

Plus, migrate from `QUnit.reset` to `QUnit.done` (see qunitjs/qunit#354).

Closes gh-2342
Ref gh-2338
@pull pull Bot added the ⤵️ pull label Mar 21, 2025
@pull pull Bot merged commit 44de3d3 into TheRakeshPurohit:main Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant