-
Notifications
You must be signed in to change notification settings - Fork 381
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
Update amphtml spec to 1bbd5505 #5998
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #5998 +/- ##
=============================================
- Coverage 80.05% 75.26% -4.80%
- Complexity 0 5703 +5703
=============================================
Files 41 218 +177
Lines 702 17275 +16573
=============================================
+ Hits 562 13002 +12440
- Misses 140 4273 +4133
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Plugin builds for bca8eb7 are ready 🛎️!
|
e0de3b2
to
7efd17f
Compare
@@ -637,6 +654,18 @@ static function () { | |||
[ 'amp-story-player' ], | |||
], | |||
|
|||
'amp_google_assistant_assistjs' => [ | |||
// Note: the first line is commented out because it is broken in AMP currently. |
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 suppose the only time this will be uncommented is when there is a future spec update that adds support for amp-google-assistant-assistjs-config > script
.
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.
Yes
Co-authored-by: Pierre Gordon <[email protected]>
Previously #5703.
./bin/amphtml-update.sh
(lando ssh -c 'bash ./bin/amphtml-update.sh vendor/amphtml'
).validator-svg.protoascii
when collecting spec data as was done previously when rules were splug out intovalidator-css.protoascii
.img[data-amp-story-player-poster-img]
.Changelog
@-moz-document url-prefix()
Firefox CSS hack into@supports (-moz-appearance:meterbar)
. Fixes Eliminate obsolete transformation of@-moz-document url-prefix()
Firefox CSS hack into@supports (-moz-appearance:meterbar)
#5801.amp-story-auto-analytics
component. This is needed by Replace amp-story-bookend with amp-story-social-share GoogleForCreators/web-stories-wp#6852 (comment).amp-story-social-share
component.amp-google-assistant-assistjs-config
component.amp-google-assistant-inline-suggestion-bar
component.amp-google-assistant-voice-bar
component.amp-google-assistant-voice-button
component.amp-ad
andamp-embed
:always-serve-npa
andblock-rtc
.amp-base-carousel
:controls
.amp-bodymovin-animation
: allowcanvas
/svg
values forrenderer
.amp-story-grid-layer
:anchor
,preset
.amp-story-page-attachment
:cta-image
,cta-image-2
.feblend
,fecomponenttransfer
,feconvolvematrix
,fediffuselighting
,fedisplacementmap
,fedistantlight
,fedropshadow
,fefunca
,fefuncb
,fefuncg
,fefuncr
,femorphology
,fepointlight
,fespecularlighting
,fespotlight
,fetile
,feturbulence
.img[data-amp-story-player-poster-img]
as child ofamp-story-poster
. Note that validity for this is currently broken by the Optimizer, to be fixed by Prevent loading=lazy attribute from being removed from amp-story-player poster img amp-toolbox-php#106.modulepreload
tolink[rel]
.kit
subdomain forfontawesome.com
font stylesheet URLs.script[amp-onerror]
. See also Add support for script amp-onerror amp-toolbox-php#83.poster
requirement foramp-video-frame
. See ✅ amp-video-iframe: Remove poster requirement amphtml#33088.Details
68e4db07...1bbd5505