Skip to content

During requestAnimationFrame detect if session has ended#1169

Merged
Manishearth merged 2 commits intoimmersive-web:mainfrom
cabanier:invalid-raf
Feb 3, 2021
Merged

During requestAnimationFrame detect if session has ended#1169
Manishearth merged 2 commits intoimmersive-web:mainfrom
cabanier:invalid-raf

Conversation

@cabanier
Copy link
Copy Markdown
Member

@cabanier cabanier commented Feb 3, 2021

@cabanier
Copy link
Copy Markdown
Member Author

cabanier commented Feb 3, 2021

@toji Closes #1168

@cabanier cabanier marked this pull request as ready for review February 3, 2021 17:36
Copy link
Copy Markdown
Contributor

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

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

r=me but remove index.html

Comment thread index.html Outdated
@@ -0,0 +1,8811 @@
<!doctype html><html lang="en">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't check in index.html

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'm used to adding it to .gitignore :-)

@cabanier
Copy link
Copy Markdown
Member Author

cabanier commented Feb 3, 2021

@Manishearth I removed index.html

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.

Post-merge LGTM, for what it's worth. Thanks!

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.

3 participants