Closed talldan closed 7 hours 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: talldan <talldanwp@git.wordpress.org>
Co-authored-by: tellthemachines <isabel_brison@git.wordpress.org>
Co-authored-by: ramonjd <ramonopoly@git.wordpress.org>
Co-authored-by: draganescu <andraganescu@git.wordpress.org>
Co-authored-by: yogeshbhutkar <yogeshbhutkar@git.wordpress.org>
Co-authored-by: jameskoster <jameskoster@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: +13 B (0%)
Total Size: 1.82 MB
Filename | Size | Change |
---|---|---|
build/edit-site/index.min.js |
219 kB | +13 B (+0.01%) |
Using scaled zoom when previewing styles was supposed to "show more" of how the style affects the look of things.
I suspect that the fact that we're in preview does not alter the fact that default scale is not showing much above the fold.
But I do agree that it looks broken at the moment.
What?
Fixes #67166
Avoids zooming out when browsing styles if the preview mode is active.
How?
Check if
isPreviewMode
is active, and if so, don't enter zoom out.I expect it should never be possible to enter zoom out from the preview mode, so part of me wonders if this should be built-in to the
useZoomOut
hook. Or alternatively (and maybe better) into thesetZoomLevel
action. Would be good to get opinions on that.Testing Instructions
Screenshots or screencast
Before
After