You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Based on ampproject/amphtml#37854 (comment) it turns out that now if you add data-amp-auto-lightbox-disable to the root html element, not only will the auto-lightbox behavior be disabled but also the auto-lightbox script will also be prevented from being autoloaded. So we need to update AMP_Auto_Lightbox_Disable_Sanitizer (introduced in #6936 for #5122) to make that change.
Acceptance Criteria
The data-amp-auto-lightbox-disable attribute should be added to the html element instead of the body element.
Implementation Brief
No response
QA Testing Instructions
No response
Demo
No response
Changelog Entry
No response
The text was updated successfully, but these errors were encountered:
Feature Description
Based on ampproject/amphtml#37854 (comment) it turns out that now if you add
data-amp-auto-lightbox-disable
to the roothtml
element, not only will the auto-lightbox behavior be disabled but also the auto-lightbox script will also be prevented from being autoloaded. So we need to updateAMP_Auto_Lightbox_Disable_Sanitizer
(introduced in #6936 for #5122) to make that change.Acceptance Criteria
The
data-amp-auto-lightbox-disable
attribute should be added to thehtml
element instead of thebody
element.Implementation Brief
No response
QA Testing Instructions
No response
Demo
No response
Changelog Entry
No response
The text was updated successfully, but these errors were encountered: