-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Comments
This is a good idea. It should go in after #1925. |
Also, I can take this one. |
we chatted in chat about this one, an interesting side effect is that we are getting closer and closer to reacts DOM events :) |
@jmeas Any news on this ? |
@jmeas I was just asking :) I know you're busy while still doing lots of open source :) |
+1 I was looking for this functionality today. |
@ianmstew feel brave enough? |
@jasonLaster Sure, I'll look into it. |
Closed by #2850 |
#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.The text was updated successfully, but these errors were encountered: