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

Allow adding of subviews #187

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

UnknownJoe796
Copy link

This fixes an issue where adding a subview to a CosmosView would cause a crash.

@evgenyneu
Copy link
Owner

Thanks for the PR. Before fixing the issue, I would like to reproduce it first. How do I reproduce the bug?

@UnknownJoe796
Copy link
Author

Oh, sorry I missed your comment in my notifications! AFAIK all you have to do is add a subview to a CosmosView and it'll crash.
I caused it working with something very similar to LifecycleHooks.
If you need me to create a test for it, I can do that.
Thanks!

@evgenyneu
Copy link
Owner

Oh I see, adding subviews to Cosmos view is not supported, to make the code simple. Sorry.

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

Successfully merging this pull request may close these issues.

2 participants