Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Fixes issue #700: pagebeforehide return 'undefined' for ui.nextPage#1190

Closed
pbarnes wants to merge 1 commit intojquery-archive:masterfrom
pbarnes:master
Closed

Fixes issue #700: pagebeforehide return 'undefined' for ui.nextPage#1190
pbarnes wants to merge 1 commit intojquery-archive:masterfrom
pbarnes:master

Conversation

@pbarnes
Copy link

@pbarnes pbarnes commented Mar 4, 2011

This commit passes a null (event) second parameter to the internal _trigger method for page 'beforeshow' and 'beforehide' methods, mimicking the 'hide' and 'show' event triggers.

Without this additional parameter, the data being passed (ui.nextPage & ui.prevPage) will never be passed to the element.trigger call and thus 'ui' will always be an empty jQ object when these two events are triggered.

@scottjehl
Copy link

Landed! Thanks.

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants