Skip to content

Use throttle instead of debounce #161

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

Closed
wants to merge 4 commits into from

Conversation

chanzxxx
Copy link
Contributor

@chanzxxx chanzxxx commented Jul 6, 2018

See detail here #160

@codecov
Copy link

codecov bot commented Jul 6, 2018

Codecov Report

Merging #161 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #161   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          92     95    +3     
  Branches       24     25    +1     
=====================================
+ Hits           92     95    +3
Impacted Files Coverage Δ
src/components/InfiniteLoading.vue 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 24aeeab...caefce2. Read the comment docs.

@PeachScript
Copy link
Owner

@whdckszxxx great work, thank you very much!

And it would be better if you remove build file updates from this PR, because it should be updated in a new version release, not a feature changes.

@chanzxxx
Copy link
Contributor Author

@PeachScript Okay I will do it later :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants