-
Notifications
You must be signed in to change notification settings - Fork 3.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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃殌 Disable FixedLayer when page is served canonically #26419
馃殌 Disable FixedLayer when page is served canonically #26419
Conversation
Don't see a way to move this to a draft. Apologies. |
*/ | ||
setup() { | ||
const viewer = Services.viewerForDoc(this.ampdoc); |
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.
Worth considering if this should return a boolean, especially since its only needed for tests.
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.
鉂わ笍
Needs @dvoytenko approval, but LGTM.
Ping @dvoytenko to review so we can merge. |
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.
Thanks!
When an AMP document is served canonically (like AMP.dev), the FixedLayer service is not needed.
This change disables the FixedLayer from performing its setup phase while we investigate a longer term fix. (#26397)