Skip to content

Deprecate AMP Layout settings #4554

@westonruter

Description

@westonruter

Feature description

The AMP Layout setting is a source of confusion:

image

Why would anyone want, for example, to set a block to have “no display”? It hides the block from the editor and from the AMP version of the page, but on the AMP version it is visible. That's not helpful. The other layout options are similarly problematic as they would apply to the AMP version of the page, and they won't display properly in the editor either.

Layout is not being exposed here at the right level of abstraction. Rather, there is no abstraction at all. The layout should be managed via the block itself seamlessly with how the block exposes its own layout options.

So I suggest that we remove the toggle entirely, unless the block currently has the setting specified. This will allow existing blocks to manipulate the layout, but for newly-added blocks the setting would go away.

We also want to minimize “AMP” anywhere in the UI. It's not something that authors should need care about at all.


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

Metadata

Metadata

Assignees

Labels

EditorEnhancementNew feature or improvement of an existing oneGroomedP1Medium priorityWS:CoreWork stream for Plugin core

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions