-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable Zoom Out if no section root to allow for Theme opt in - 6.7 #68229
base: wp/6.7
Are you sure you want to change the base?
Conversation
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 If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Size Change: +38 B (0%) Total Size: 1.77 MB
ℹ️ View Unchanged
|
I would like to discuss whether this PR should be shipped to WP 6.7.2. This PR disables zoom out mode when there is no section root. This means that in the post editor, zoom out mode will be disabled regardless of whether it is an iframe editor or not, which will resolve the issue reported in #66884. However, I'm concerned that making such a change in a minor release may have a bit of a big impact. |
I also left a comment on the PR: #67232 (comment). This change makes the Zoom out feature's availability inconsistent. The conditions are implicit and not communicated with the end user. P.S. The Zoom out feature gets enabled for posts if you add a group and set the element as |
What?
Backports #67232 to
wp/6.7
for 6.7.2 release.