Skip to content

Explicitly specify that reset events work on offset spaces#1024

Merged
toji merged 1 commit intoimmersive-web:masterfrom
Manishearth:reset-offset
May 11, 2020
Merged

Explicitly specify that reset events work on offset spaces#1024
toji merged 1 commit intoimmersive-web:masterfrom
Manishearth:reset-offset

Conversation

@Manishearth
Copy link
Copy Markdown
Contributor

@Manishearth Manishearth commented May 11, 2020

I considered writing this out as steps, but I realized the reset event really has one step and all the complexity is in the conditions when it gets triggered.

Fixes #933


Preview | Diff

@Manishearth Manishearth requested a review from toji May 11, 2020 20:56
Copy link
Copy Markdown
Member

@toji toji left a comment

Choose a reason for hiding this comment

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

LGTM!

@toji toji merged commit fd033e5 into immersive-web:master May 11, 2020
@Manishearth Manishearth deleted the reset-offset branch May 11, 2020 21:46
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.

Clarify that offset spaces inherit reset events and bounds changes

2 participants