Open getdave opened 3 days ago
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot
label.
If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
Co-authored-by: getdave <get_dave@git.wordpress.org>
Co-authored-by: jeryj <jeryj@git.wordpress.org>
Co-authored-by: draganescu <andraganescu@git.wordpress.org>
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.
Size Change: +52 B (0%)
Total Size: 1.82 MB
Filename | Size | Change |
---|---|---|
build/block-editor/index.min.js |
255 kB | +29 B (+0.01%) |
build/editor/index.min.js |
112 kB | +23 B (+0.02%) |
Something like this? https://github.com/WordPress/gutenberg/pull/67245/files
@jeryj I replied on your PR.
What?
Disable Zoom Out if no section root to allow for Theme opt in by defining a `z
Related to https://github.com/WordPress/gutenberg/pull/67113.
Why?
Because reports have been coming in that the block layout of some Themes isn't compatible with a good experience in Zoom Out.
How?
With this change if a
<main>
tag is not defined (or we're not editing a entity with acore/post-content
block) then Zoom Out will be disabled.This allows Themes to opt in to Zoom Out based on where they place a
<main>
tag. If no<main>
is defined then Zoom Out will not work.Testing Instructions
<main>
from templateTesting Instructions for Keyboard
Screenshots or screencast