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.

Sign up for GitHub

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

Jump to bottom

Don't trigger event for hidden elements #36

Closed

caseyWebb wants to merge 1 commit into protonet:master from caseyWebb:hidden-elements

caseyWebb commented Nov 18, 2014

Passes all tests

[ref] #6


          Don't trigger event for hidden elements

0a143d7

[ref] #6

robertfausk commented Jan 23, 2015

@tiff What about merging this PR? I would really appreciate this one.

Member

tiff commented Jan 26, 2016

this would not trigger the inview event for elements that have zero height (since :visible is based on that). not sure if I want this :)

caseyWebb closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet