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

few issues #17

Open
qiuzh opened this issue Oct 7, 2011 · 0 comments
Open

few issues #17

qiuzh opened this issue Oct 7, 2011 · 0 comments

Comments

@qiuzh
Copy link

qiuzh commented Oct 7, 2011

Hi ,

I’m using the ajaxmanager, it is very help for me, thank you very much.
When I use it found below things, I appreciate if you can consider them and make some change in next version.

  1. _removeXHR function

In this function will call $.dequeue first then call inprogress--;
And $.dequeue will execute a _createAjax function to increase the inprogress.
If lots of ajax in the queue then will keep increase inprogress untill no ajax in the queue then
_removeXHR got chance to call inprogress--;

Idealy if 5 ajax in the queue, the inprogress should change as 1 0 1 0 1 0 1 0 1 0
But now inprogress change as 1 2 3 4 5 4 3 2 1 0

*I thinks should call inprogress--; before call $.dequeue
2. getData function
in this function got below condition Got below condition
if(!ret && this.opts.queue)
is it necessary to use this.opts.queue ?
Sometime I need change the queue as false for a while,if I do so the abort function can not work because this getData function can not return the right function

*I thinks maybe can remove && this.opts.queue

3._createAjax function

In this function if(o.cacheResponse && cache[id]) it will simulate the ajax success and complete
I think if o.beforeCreate return false also need simulate the complete function
So that we can got a chance to do something in that function when beforeCreate return false

4.actually what I really need is a gap between ajax functions
now if lots of ajax in the queue already, the script will keep dequeue and process the ajax, then the UI got no chance to process other tasks(like detect user action or show some animation) so the the page performance will not good.
so i think it is better if have a gap after one ajax complete to release the CPU for other tasks

My issues quite messy hope you can understand what i want to say.
again thank you for you provide so good plugin.

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

No branches or pull requests

1 participant