Converted to a jQuery plugin #11
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…l configurable still need to add AJAX and css transitions
…ck to the slideUp callback Will be changing this so can use css3 transitions/animations instead of jQuery animation. Still need to work out the bug where callback is firing multiple times. Firefox does not like to reload page until mouse has scrolled down first. May use mouse movements in addition to touch and mousewheel
…is only fired once.
Updated readme for current know issues. Updated assets to use hook as spinner, using css3 animations to spin it.
…dded min file Also added my name to authors.
|
Just a thought, could this be used as "Pull to Load More" at the bottom of a listview? An option to do so would be cool. |
|
@gogobu there is a callback called reloadEl that instead of reloading the page will allow you to do run any other javascript you would like, like loading in more HTML. |
|
Sorry I wasn't clear enough. What I meant was instead of pull from the top of the page, a "pull to load more" at the bottom of the page to load more list items. (smile) |
jordansinger
added a commit
that referenced
this pull request
Apr 19, 2013
Converted to a jQuery plugin
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These commits change the script to a jQuery plugin to make Hook.js more extendable. I also changed the spinner from a gif to the Hook logo. I'm using css3 animations to spin the logo. Can certainly remove this, and resend pull request if needed. Also the plugin will require the use of mousewheel.js if developer wants support for mouse wheel. Still need to update for using css transitions instead of animate, but this at least a start. Since there is a callback this should fix issue #8. There is no Safari detection so fixes Issue #7. Fixes #6 the infinite reloading through using "window.location.reload(true);". Should also take care of some of the issues in #9 since it supports touch devices and mouse wheel.