-
Notifications
You must be signed in to change notification settings - Fork 168
bug fixed & enhancement #33
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
Conversation
@jsdf I'm not sure how to pass travis-ci/pr |
Can we get this merged? |
That's the plan |
Okay so I finally got some time to pull down this branch and take a look at it, and I think some of the new features need a bit of polish but I'm mindful that people want the bug fixes so I'm just going to merge those for now. |
@jsdf any news on this ? |
Sorry I'm just on holiday at the moment, back this weekend
|
@jsdf no problem, enjoy |
@atom2ueki If you have time, it might be worth separating the bug fix from the enhancement. That way @jsdf can focus on reviewing and merging the bug fix with higher priority (since presumably it involves less code) and then give feedback on the enhancement when he has time. Just my $0.02 |
Ah it's okay, I'm going to fix both at the same time (though with a
|
bug fixed
bug fixed for
#32
#27
enhancement - New props/attributes added
pullingIndicator: oneOfType([func, element])
- define the indicator shown when pulling down ( optional )holdingIndicator: oneOfType([func, element])
- define the indicator shown when pulling down and holding after it exceed the min-distance ( optional )refreshingIndicator: oneOfType([func, element])
- define the indicator shown when refreshing ( optional )pullingPrompt: String
default 'pull to refresh'holdingPrompt: String
default 'release to refresh'refreshingPrompt: String
default 'loading'new example added
please refer to this example to learn how to use the version of this PR