Update AMP settings to include additional options

[jwold]

Feature description

• Remove "redirect mobile visitors to AMP" toggle from onboarding
• Add new section in AMP Settings under Advanced Settings to "redirect mobile visitors to AMP". Turn on by default.
• Add new "dev tools" toggle in advanced settings. Make note that it's user role specific (not site wide)

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Implementation brief

QA testing instructions

Demo

Changelog entry

[westonruter]

See also #5386.

[jwold]

[westonruter]

@jwold IMO, the headings under the Other section for “Redirects” and “Dev Tools” are too similarly-sized to the toggles they are for. Maybe the headings are redundant?

[jwold]

@westonruter that's a good point. That heading level has always been a bit weird. I just submitted an attempted fix. Feel free to remove it if my code was incorrect; figured I'd try! 😀

Screenshot of what I'm trying to do: https://d.pr/i/CPtRce

• Commit 1
• Commit 2 [fixing tab issue]

[bartoszgadomski]

QA passed

✅   Add new section in AMP Settings under Advanced Settings to "redirect mobile visitors to AMP". Turn on by default.

The "redirect mobile visitors to AMP" toggle is found under the "Other" section when site is in "Transitional" or "Reader" mode:

I can also see that mobile_redirect option is set to true by default.

✅   Remove "redirect mobile visitors to AMP" toggle from onboarding

All good, I did not found this option in onboarding wizzard.

✅   Add new "dev tools" toggle in advanced settings. Make note that it's user role specific (not site wide)

All good, "Enable developer tools" toggle found under "Other" section: