Skip to content

✅ Add validation for rotate-to-fullscreen in amp-video#14969

Merged
alanorozco merged 2 commits intoampproject:masterfrom
alanorozco:validate-video
May 1, 2018
Merged

✅ Add validation for rotate-to-fullscreen in amp-video#14969
alanorozco merged 2 commits intoampproject:masterfrom
alanorozco:validate-video

Conversation

@alanorozco
Copy link
Copy Markdown
Member

Adding only for amp-video as audit for support in other players is pending.

Copy link
Copy Markdown
Contributor

@aghassemi aghassemi left a comment

Choose a reason for hiding this comment

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

@alanorozco
Copy link
Copy Markdown
Member Author

@aghassemi done.

@aghassemi
Copy link
Copy Markdown
Contributor

LGTM

@alanorozco alanorozco force-pushed the validate-video branch 6 times, most recently from f3eb0bd to 3671ca1 Compare May 1, 2018 17:15
@alanorozco alanorozco merged commit eafecaa into ampproject:master May 1, 2018
@alanorozco alanorozco deleted the validate-video branch May 1, 2018 17:51
noranazmy pushed a commit to noranazmy/amphtml that referenced this pull request May 10, 2018
honeybadgerdontcare added a commit that referenced this pull request May 10, 2018
* Revision bump for #14969

* Revision bump for #14937

* Simplify the script tag error.

* Whitelist pro.fontawesome.com in the AMP font list.

* Error category remapping

* Revision bump for #15106

* Revision bump for #15129

* Revision bump for #15155/15164/15191/15187
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants