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: Stringdefault 'pull to refresh'holdingPrompt: Stringdefault 'release to refresh'refreshingPrompt: Stringdefault 'loading'new example added
please refer to this example to learn how to use the version of this PR