Skip to content
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

add isAutoDestory,and refactor the code #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zzuligy
Copy link

@zzuligy zzuligy commented Jul 4, 2018

添加自动销毁,当数据量大的时候可以防止页面崩溃。

重构代码,去掉容易的变量 isInit,去掉逻辑冗余requestAnimationFrame(vue 内部机制watch queue已经等价,并且requestAnimationFrame 的内部setTimeout实现也有悖于任务队列的基本原理)

个人见解,可能有误,欢迎指正

add autoDestory, so cope with the big data.

refacte and remove the redundant code.

I think the requestAnimationFrame method is useless,because vue has do the job.and the setTiemout
in requestAnimationFrame is mismatch the job queue of js.

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.

1 participant