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

YouTube Video Not Displaying Full Screen Correctly #1646

Closed
nuclearghost opened this issue Jan 28, 2016 · 6 comments
Closed

YouTube Video Not Displaying Full Screen Correctly #1646

nuclearghost opened this issue Jan 28, 2016 · 6 comments
Assignees
Milestone

Comments

@nuclearghost
Copy link
Contributor

On Mac OS X 10.11.2, Chrome Version 47.0.2526.111 (64-bit)

When trying to watch the YouTube video embedded on this page https://www.ampproject.org/docs/get_started/about-amp.html the video doesn't actually go full screen.

Here's a screenshot of what the video looks like in full screen.
screen shot 2016-01-28 at 1 47 45 pm

Full screen YouTube works fine on the main site and other embeds on third party sites pointing toward an issue with the amp-youtube element.

@dvoytenko
Copy link
Contributor

/cc @cramforce @pbakaus @Meggin

This is not due to amp-youtube. I just replaced it with a simple youtube iframe and the problem persists. I believe this is due to the following CSS on the page:

.main .content {
  will-change: transform;
}

My best guess is this is some sort of a bug and/or conflict with layering for hardware acceleration. A bug to Chrome is likely due. PTAL.

@cramforce
Copy link
Member

@dvoytenko Do you want to file. Very nice find!

@dvoytenko
Copy link
Contributor

I guess I should copy the whole document into JSBIN before it gets fixed...

@dvoytenko
Copy link
Contributor

@cramforce cramforce assigned pbakaus and unassigned mkhatib Jan 29, 2016
@cramforce
Copy link
Member

Cool bug. I assigned this one to @pbakaus because I think that is his CSS. Might be mistaken, though.

@rudygalfi rudygalfi modified the milestones: M2, M1 Feb 4, 2016
@rudygalfi rudygalfi modified the milestones: M2, Backlog Mar 4, 2016
@adelinamart
Copy link
Contributor

Hey,

The AMP community has been working nonstop to make AMP better, but somehow we've still managed to grow an enormous backlog of open issues. This has made it difficult for the community to prioritize what we should work on next.

A new process is on the way and to give it a chance for success we will be closing issues that have not been updated in awhile.

If this issue still requires further attention, simply reopen it. Please try to reproduce it with the latest version to ensure it gets proper attention!

We really appreciate the contribution! Thank you for bearing with us as we drag ourselves out of the issue abyss. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants