-
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
feat: Rename Reusable blocks to Patterns #51704
Conversation
10a84f7
to
609f58c
Compare
Size Change: 0 B Total Size: 1.44 MB ℹ️ View Unchanged
|
Flaky tests detected in 3407ab3f30b193976ea79f37027ba6c8234a3f74. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5334708915
|
@glendaviesnz will you please confirm the labels in the screenshots align with your expectations based upon #51144? For context, the unsupported block label is controlled by the block name, so it is somewhat unrelated to the proposed "Synced patterns" changes. gutenberg/packages/block-library/src/missing/edit.native.js Lines 150 to 151 in 3257238
|
Looks correct to me. Thanks. |
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.
LGTM! Thank you for updating these 🚀
I've tested these new changes on an iPhone 14 Pro simulator using the main apps.
@glendaviesnz is there a likelihood we will revert the web string changes? I saw conversations challenging the web change. If reverting is unlikely, I will merge this before tomorrow's mobile release cut. |
I have only just seen those, maybe it wouldn't hurt to hold off for a few days and see where that discussion goes. |
Align with new web editor terminology: 487f230
Align with new web editor terminology: 487f230
3407ab3
to
6d179fa
Compare
After 7 days, there appears to be little movement on discussions challenging the rename. Elsewhere, work continues to update the naming. Enabling auto-merge for this now. |
Related PRS
What?
Rename "Reusable blocks" to "Synced patterns."
Why?
Align with new web editor terminology: 487f230
How?
Update UI text strings and test IDs.
Testing Instructions
without converting the pattern to regular blocks.
Testing Instructions for Keyboard
n/a
Screenshots or screencast