-
Notifications
You must be signed in to change notification settings - Fork 26
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
Access to "view" #39
Comments
Care to explain why you need access to |
Sure, it's because of the Blaze.remove() as mentioned here #36 . I wanted to be helpfull and group some feedback in the Q&A thread, but as I respond now I guess it's making a bit of a mess of the issues :). So plz remove if needed. |
I still do not get you. You render manually and you cannot keep the |
Yes, but just to be able to pass it on to Blaze.remove(). I use componentChildren() to find out all existing "panels" inside a viewstack component of mine. Having the component works, than just need to get the view of that component to be able to remove it once I click a "close" like button from within that panel. |
Hm, should this be |
instance.remove() has a nice feel to it. Makes more sense to me than BlazeComponent.remove() which only has going for it that it matches Blaze, but Blaze has no (apparent) parent/child relationships to factor in (as far as I know). Calling remove on the parent is more specific. |
On reading it again, I didn't get your comment correctly. There is one more (which I thought you meant):
|
Yes. I thought more like Good idea to remove children. |
But probably it is better to remove itself, because you can always first access the child though |
true. sounds good! |
Implemented as |
Acces to "view": I'm already having trouble with everything related to views/templates/templateinstances and what not, I hoped the components wouldn't add to that list but abstract it all away. That maybe coming in the future. But until then I found that the only way to access the view the component is related to was through _componentInternals. Maybe something like a "getView" on the component? Preference is to abstract it away though :)
The text was updated successfully, but these errors were encountered: