-
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
Patterns: Add duplicate pattern command #55292
Conversation
Size Change: +362 B (0%) Total Size: 1.65 MB
ℹ️ View Unchanged
|
3b1e990
to
e4eda2d
Compare
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 tested well for me:
✅ Duplicate pattern command was only available on pattern summary and edit pages and worked as expected
✅ Theme pattern categories were duplicated via the labels and not slugs for both duplicate menu item and duplicate command
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.
Works well for me! Do you think if it requires more work if we support the duplicating template parts command too?
Thanks for the reviews!
Commands for templates and template parts were going to be a follow-up. I noted that in #55188 which this PR originally depended on but missed adding it to the PR description here. I'll get this merged and begin follow-up on template parts shortly. Edit: I've added a note to the PR description about template parts being out of scope here. |
Related:
What?
Extracts pattern duplication modal into the patterns package and adds a new duplicate pattern command for the site editor.
Note: Commands for templates and template parts will be considered separate follow-ups
Why?
Makes it easier for users to work around the fact they cannot change a pattern's sync status.
How?
Testing Instructions
Screenshots or screencast
Screen.Recording.2023-10-12.at.5.23.30.pm.mp4