-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Youtube theater mode not using full screen width #13232
Comments
This is a version of #11574 -- it looks like Firefox is dynamically resizing its viewport depending on the presence/absence of the dynamic toolbar. You can see this by visiting https://www.whatismybrowser.com/ and comparing the "Browser Window Size" between Firefox vs. Chrome/Safari (on iOS). For me on my iPad, Firefox reports 748px vs. Safari/Chrome report 768px (when I'm holding my tablet in landscape mode). So the root issue here is that Firefox is using a shorter viewport here than Chrome and Safari (and then YouTube sees that the viewport is shorter and it makes the video a bit smaller, because they have some logic to ensure that the video plus a bit of the content that follows it is visible in the viewport-height-that-the-browser-reports-to-them). |
Given that the iOS version of Chrome can handle vh units properly, there should be APIs to tell the dynamic toolbar initial height to WebKit and tell the intermediate height to WebKit as well. |
Hello! Thanks for your input, I believe this relates to #11574 then. |
This issue has been automatically marked as stale. Has the issue been fixed, or does it still require the community's attention? Please leave any comment to keep this issue opened. It will be closed automatically if no further update occurs in the next 30 days. Thank you for your contributions! |
Still reproducible as-described in initial comment. (I just retested with current Firefox release vs. Safari, on my iPad.) |
Steps to reproduce
Expected behavior
Actual behavior
Device & build information
Attachments:
┆Issue is synchronized with this Jira Task
The text was updated successfully, but these errors were encountered: