-
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
Tweak, and add, more consistent commands #53496
Conversation
Size Change: +1.38 kB (0%) Total Size: 1.5 MB
ℹ️ View Unchanged
|
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.
Left a small comment otherwise, this LGTM
9f3de64
to
afe81c6
Compare
packages/edit-site/src/hooks/commands/use-edit-mode-commands.js
Outdated
Show resolved
Hide resolved
Flaky tests detected in afe81c6. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5825553707
|
Co-authored-by: Riad Benguella <[email protected]>
It looks like this PR introduced a duplicate Patterns command. One already existed called "Open patterns" and took into account whether or not the active theme could access the site editor. See #52817 for the PR introducing the original command. |
The duplicate patterns commands have been consolidated in #54133. |
What?
Why?
Trying to establish consistency with command language, style, and actions.
Testing Instructions
Testing Instructions for Keyboard
Visuals
Before
After