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
Add WebXR settings and permission flow #3019
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should not show the icon for the content feed but looks good otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM although I notice we need to do mulit-lang support for WebXR permission page. We can open another bug to work on it.
What do you mean by multi-lang? The localization happens automatically after the PR lands. |
Yep. That's what I mean. It would be great to have the localization automatically. |
Hiding the icon for the feed can probably be a follow up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you have a window opened with a webxr blocked site and you clear the blocked sites in settings, the URL bar is not updated and the site is not reloaded.
I'm going to land and leave this as a follow up.
Fixes #2062