-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Zoom Out: The patterns tab behind a new experiment. #61601
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: +1.08 kB (+0.06%) Total Size: 1.75 MB
ℹ️ View Unchanged
|
In another channel @scruffian and I thought it should be better to create an experiment rather than having to use the global styles browser to trigger zoom out mode, that is until the assembly experience is ready for prime time. |
This pull request has changed or added PHP files. Please confirm whether these changes need to be synced to WordPress Core, and therefore featured in the next release of WordPress. If so, it is recommended to create a new Trac ticket and submit a pull request to the WordPress Core Github repository soon after this pull request is merged. If you're unsure, you can always ask for help in the #core-editor channel in WordPress Slack. Thank you! ❤️ View changed files❔ lib/experimental/editor-settings.php ❔ lib/experiments-page.php |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This tests as described, left a small copy nit.
Co-authored-by: Andrei Draganescu <me@andreidraganescu.info>
Flaky tests detected in bc48a5c. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/9161185123
|
* Zoom Out: Remove from the patterns tab until we're ready * hide the pattern tab zoom out behind an experiment * Update lib/experiments-page.php Co-authored-by: Andrei Draganescu <me@andreidraganescu.info> --------- Co-authored-by: Andrei Draganescu <me@andreidraganescu.info> Co-authored-by: scruffian <scruffian@git.wordpress.org> Co-authored-by: draganescu <andraganescu@git.wordpress.org>
What?
In trunk we zoom out the canvas when you open the patterns tab. This PR hides that behaviour behind an experimental flag.
Why?
I don't think the current zoom out experience is good enough to be useful yet, so we should remove this from the next version of Gutenberg until it's ready.
How?
Put the hook behind an experimental flag.
Testing Instructions
Note
Zoom out can still be accessed from Global Styles > Browse Styles.