Skip to content

Update sanitizer for auto-lightbox disabling to target root element instead of body #7057

Closed
@westonruter

Description

@westonruter

Feature Description

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    BugSomething isn't workingChangeloggedWhether the issue/PR has been added to release notes.

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions