Skip to content

Extend Sandboxing experiment to Paired AMP modes #7268

@westonruter

Description

@westonruter

Feature Description

Currently the Sandboxing experiment is limited to Standard mode. This is because pages using the experiment are not valid AMP, meaning they will not get indexed. Nevertheless, invalid paired AMP pages are still useful for one important case: a mobile-optimized version of the site. When mobile redirection is enabled in particular, being able to serve pages using the AMP runtime and our optimizations (e.g. Reader theme) will enable the plugin to be usable as a "mobile theme" plugin.

  • Show the Sandboxing experiment panel when any template mode is selected on the Settings screen, not just when the Standard mode is selected.
  • When a non-strict sandboxing level is selected, never output the link[rel=amphtml] element but always output link[rel=alternate] which is output when mobile redirection is enabled. The reason here is that we do not want to advertise to Googlebot in this case that there is an AMP page because then it will complain if there are validation errors.
  • Prevent redirecting from AMP to non-AMP when in a paired mode if the Sandboxing level is not set to Strict (and there is kept invalid markup).
  • Show link to exit mobile version on AMP pages even when mobile redirection is not enabled  #5293

Acceptance Criteria

No response

Implementation Brief

No response

QA Testing Instructions

No response

Demo

No response

Changelog Entry

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    ChangeloggedWhether the issue/PR has been added to release notes.EnhancementNew feature or improvement of an existing one

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions