-
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
Merge #63002 into wp/6.6 - Site Editor: patterns and templates cannot be edited from sidebar mobile view #63055
Merge #63002 into wp/6.6 - Site Editor: patterns and templates cannot be edited from sidebar mobile view #63055
Conversation
…ile view (#63002) * In mobile mode, check for canvas edit mode so the "Edit" callback redirect the browser to the block editor in edit mode * I suspect that the transition is causing intermittent crashing in Chrome in mobile viewports. Turning it off as it appears more stable in local testing. * Revert unintentional change Co-authored-by: ramonjd <ramonopoly@git.wordpress.org> Co-authored-by: andrewserong <andrewserong@git.wordpress.org> Co-authored-by: youknowriad <youknowriad@git.wordpress.org> Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: priethor <priethor@git.wordpress.org> Co-authored-by: annezazu <annezazu@git.wordpress.org> Co-authored-by: ironprogrammer <ironprogrammer@git.wordpress.org> Co-authored-by: ivan-ottinger <ivanottinger@git.wordpress.org> Co-authored-by: mrfoxtalbot <mrfoxtalbot@git.wordpress.org>
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: +5 B (0%) Total Size: 1.75 MB
ℹ️ View Unchanged
|
Flaky tests detected in 2cee795. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/9760391033
|
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.
Thanks for the fix. I can repro the bug in 6.6 and this fixes it.
What?
Merging #63002 into
wp/6.6
#63002 addressed the following:
Note
This PR omits the changes added in #62705 if you were wondering why the code is different to #63002
Why?
There were merge conflicts, mainly due to #62705 not featuring in the
wp/6.6
branch.Testing Instructions
Taken from #63002
782px
wide to trigger the mobile viewThis branch in action
2024-07-02.21.48.09.mp4