Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add pageshow and pagehide #2701

Merged
merged 1 commit into from Feb 12, 2015
Merged

Conversation

@SergioCrisostomo
Copy link
Member

SergioCrisostomo commented Feb 12, 2015

fixes #2684

We could also add

if (type == 'pageshow' || type == 'pagehide'){
    this.persisted = event.persisted;
}

in DOMEvent.js, but since the original event is always accessible in e.event maybe no need?

@SergioCrisostomo SergioCrisostomo changed the title add pageshow and page hide add pageshow and pagehide Feb 12, 2015
arian added a commit that referenced this pull request Feb 12, 2015
add pageshow and pagehide
@arian arian merged commit a1e68c7 into mootools:master Feb 12, 2015
1 check was pending
1 check was pending
continuous-integration/travis-ci The Travis CI build is in progress
Details
@SergioCrisostomo SergioCrisostomo deleted the SergioCrisostomo:pagehide_pageshow branch Feb 12, 2015
@SergioCrisostomo SergioCrisostomo restored the SergioCrisostomo:pagehide_pageshow branch Mar 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.