Skip to content

🐛 Remove RESPONSIVE from amp-story-360 validator#30869

Merged
processprocess merged 1 commit intoampproject:masterfrom
processprocess:360-layout
Oct 27, 2020
Merged

🐛 Remove RESPONSIVE from amp-story-360 validator#30869
processprocess merged 1 commit intoampproject:masterfrom
processprocess:360-layout

Conversation

@processprocess
Copy link
Contributor

layout=responsive is incompatible with the component.
This appears to be due to the use of applyFillContent to the wrapper of the canvas element.

cc @ampproject/wg-stories

@CLAassistant
Copy link

CLAassistant commented Oct 26, 2020

CLA assistant check
All committers have signed the CLA.

@amp-owners-bot
Copy link

Hey @gmajoulet, @t0mg! These files were changed:

extensions/amp-story-360/validator-amp-story-360.protoascii

@gmajoulet
Copy link
Contributor

I created a cookbook query to validate that this will not make any story invalid and will likely LGTM tomorrow.

@gmajoulet
Copy link
Contributor

cc @ampproject/wg-caching cookbook 425085

@processprocess processprocess merged commit a8a503b into ampproject:master Oct 27, 2020
caoboxiao added a commit to caoboxiao/amphtml that referenced this pull request Oct 28, 2020
@caoboxiao caoboxiao mentioned this pull request Oct 28, 2020
caoboxiao added a commit that referenced this pull request Oct 29, 2020
* cl/338138658 Allow blurry image placeholders for Transformed AMP

* cl/338723089 Revision bump for #30832

* cl/338726670 Remove tailing whitespaces

* cl/339321880 Revision bump for #30847

* cl/339356382 Revision bump for #30869

Co-authored-by: honeybadgerdontcare <sedano@google.com>
@processprocess processprocess deleted the 360-layout branch November 12, 2020 17:58
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
* cl/338138658 Allow blurry image placeholders for Transformed AMP

* cl/338723089 Revision bump for ampproject#30832

* cl/338726670 Remove tailing whitespaces

* cl/339321880 Revision bump for ampproject#30847

* cl/339356382 Revision bump for ampproject#30869

Co-authored-by: honeybadgerdontcare <sedano@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants