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

Only allow pointer events on children of active layers. #802

Merged
merged 2 commits into from
Mar 30, 2018

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Mar 29, 2018

This allows multiple active layers to work together. Before, the topmost active layer will receive all pointer events.

This also fixes an issue in the sceneObject exit function (it should clear the parent, not delete the parent function). This, in turn, required a fix to the annotation test because the test map is destroyed between tests.

This allows multiple active layers to work together.  Before, the
topmost active layer will receive all pointer events.

This also fixes an issue in the sceneObject exit function (it should
clear the parent, not delete the parent function).  This, in turn,
required a fix to the annotation test because the test map is destroyed
between tests.
@aashish24
Copy link
Member

interesting that we did not catch this before. I am wondering how did you find about this?

Copy link
Member

@aashish24 aashish24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@manthey
Copy link
Contributor Author

manthey commented Mar 30, 2018

@matthewma7 reported this when he created two UI layers. If we have only one UI layer, the issue wasn't obvious.

@manthey manthey merged commit 39cd0f5 into master Mar 30, 2018
@manthey manthey deleted the layer-pointer-events branch March 30, 2018 12:24
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