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.

Sign up for GitHub

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

Jump to bottom

Tree expand should return promise #739

Closed

mvaerle opened this issue Oct 3, 2013 · 0 comments

mvaerle commented Oct 3, 2013

I'm using a JsonRest store in combination with the tree plugin. Expanding a row triggers a request, and I want to do something in response to this.

There is already a Deferred being created in expand, however it is not returned by expand. See the following gist for what a patch could look like:

https://gist.github.com/mvaerle/6810641

edhager added a commit to edhager/dgrid that referenced this issue


          Fixes dojo#739, tree.expand now returns a promise.

c8f200f

edhager mentioned this issue

Make tree expand return a promise. #815

Closed

edhager closed this as completed in

dbcb6d0

mvaerle mentioned this issue

Async expand computes incorrect scrollHeight leading to rows not displaying #904

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment