-
Notifications
You must be signed in to change notification settings - Fork 383
Description
Feature description
In the theme of preventing the user from having to know anything about AMP (#4554, #4555, #4556, #4557) to author content, we should also consider eliminating the dedicated “AMP Settings” panel from the Gallery block:
At one level this could just be a matter of removing “AMP” from the panel title. It could also mean moving the toggles under “Advanced”.
We also need to take into account how other plugins extend the Gallery block. Jetpack for example has this setting:
When that is enabled (as of Jetpack 8.7) the effect is as if the two toggles in AMP settings have been turned on. Users will be confused as to what purpose they have when this Jetpack feature is enabled.
Perhaps the AMP-provided settings panel should be made easy for another plugin to turn off (e.g. Jetpack).
We should also consider whether this functionality belongs in the AMP plugin in the first place, and if it should be removed/deprecated in favor of ecosystem plugins (like Jetpack).
Jetpack's implementation is key because it provides a non-AMP version on non-AMP pages. The AMP plugin doesn't currently provide that, which is yet another source of confusion for users. If a site is in a paired AMP setup (Reader/Transitional mode), why should there be AMP-specific settings present in the first place if functionality won't be working in the canonical (non-AMP) version of the page?
We should limit extending blocks with AMP functionality when that functionality can be used on any version of the page. This will be greatly facilitated by Bento AMP. However, in the meantime, we should consider whether this functionality should be in the AMP plugin or relegated to the ecosystem.
Do not alter or remove anything below. The following sections will be managed by moderators only.

