Skip to content
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

Default pattern terminology to Pattern instead of Block Pattern #54708

Open
wants to merge 5 commits into
base: trunk
Choose a base branch
from

Conversation

foosantos
Copy link
Member

What?

This PR replaces mentions of "Block Patterns" with "Patterns" only.

Fixes #49617.

Why?

There's an inconsistency in the labeling of the Patterns — sometimes it's "Pattern" while sometimes it's "Block Patterns" — and the goal would be to fix that.

How?

I mostly searched for the mentions of "Block Patterns" and replaced them manually based on the context.

Testing Instructions

Basically, a good step would be to search for mentions of patterns in the code and see if makes sense for these contexts.

@github-actions github-actions bot added the First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository label Sep 22, 2023
@github-actions
Copy link

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @foosantos! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other.

If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information.

@foosantos foosantos added the [Type] Enhancement A suggestion for improvement. label Sep 22, 2023
README.md Show resolved Hide resolved
@foosantos foosantos requested a review from richtabor October 11, 2023 16:18
Copy link
Contributor

@draganescu draganescu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@t-hamano
Copy link
Contributor

t-hamano commented Nov 7, 2023

@foosantos

In order to merge this PR, could you rebase with the latest trunk just to be sure? If the GitHub Action fails, you'll probably need to fix that too. thank you!

@Mamaduka Mamaduka removed their request for review March 26, 2024 09:12
@ObliviousHarmony ObliviousHarmony removed their request for review August 26, 2024 21:26
@dcalhoun dcalhoun removed their request for review August 28, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository [Type] Enhancement A suggestion for improvement.
Projects
No open projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

Consistent terminology: Patterns vs. Block Patterns
4 participants