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 "before:detach" / "detach" events? #1964

Closed
paulovieira opened this issue Oct 4, 2014 · 10 comments
Closed

Add "before:detach" / "detach" events? #1964

paulovieira opened this issue Oct 4, 2014 · 10 comments

Comments

@paulovieira
Copy link
Contributor

#1957 adds the "before:attach"/"attach" events for the tree view that is placed in the DOM (using region's show). One common case for this event is to activate 3rd party plugins located deep in the view tree and that must be started only after the view is in the DOM.

Should we add "before:detach" / "detach" events as well? These might be necessary to shut down those plugins.

In principle it could (and should) be done on the view's "destroy". But if preventDestroy is true, beforeDestroy/destroy will not be fired.

@jamesplease
Copy link
Member

This is a good idea. It should go in after #1925.

@jamesplease jamesplease added this to the v2.3 milestone Oct 4, 2014
@jamesplease jamesplease self-assigned this Oct 4, 2014
@jamesplease
Copy link
Member

Also, I can take this one.

@samccone
Copy link
Member

samccone commented Oct 4, 2014

we chatted in chat about this one, an interesting side effect is that we are getting closer and closer to reacts DOM events :)

@stephanebachelier
Copy link
Contributor

@jmeas Any news on this ?

@jamesplease
Copy link
Member

Nope. I recall thinking at some point that #1925 landing first would make this easier, but I forget the specifics of why. My plan of attack was to work on #1925 first, but I've been busy.

@stephanebachelier
Copy link
Contributor

@jmeas I was just asking :) I know you're busy while still doing lots of open source :)

@jamesplease jamesplease modified the milestones: v2.4, v2.3 Dec 18, 2014
@cmaher cmaher modified the milestones: v2.4, v2.5 Feb 24, 2015
@marcusstenbeck
Copy link

+1

I was looking for this functionality today.

@jasonLaster
Copy link
Member

@ianmstew feel brave enough?

@ahumphreys87 ahumphreys87 modified the milestones: v3.0.0, v2.5 Apr 12, 2015
@ianmstew
Copy link
Member

@jasonLaster Sure, I'll look into it.

@jasonLaster jasonLaster modified the milestones: v3.0.0, v3.x Aug 18, 2015
@ianmstew ianmstew modified the milestones: v3.0.0, v3.x Sep 19, 2015
@ianmstew ianmstew assigned ianmstew and unassigned jamesplease and ianmstew Sep 19, 2015
@paulfalgout
Copy link
Member

Closed by #2850

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

No branches or pull requests

10 participants