Skip to content

Editor: Define the labels of the pattern categories taxonomy#5929

Closed
youknowriad wants to merge 1 commit intoWordPress:trunkfrom
youknowriad:update/pattern-categories-labels
Closed

Editor: Define the labels of the pattern categories taxonomy#5929
youknowriad wants to merge 1 commit intoWordPress:trunkfrom
youknowriad:update/pattern-categories-labels

Conversation

@youknowriad
Copy link
Copy Markdown
Contributor

@youknowriad youknowriad commented Jan 23, 2024

Trac ticket: https://core.trac.wordpress.org/ticket/60322

In WordPress 6.5, the editor UI is going to be using a standard component for the taxonomies in the editor including the pattern categories. This means that all the labels need to be defined properly for this taxonomy.

This backports the change from WordPress/gutenberg#57094

Copy link
Copy Markdown
Contributor

@ntsekouras ntsekouras left a comment

Choose a reason for hiding this comment

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

✅ Looks good, thank you!

@youknowriad
Copy link
Copy Markdown
Contributor Author

@youknowriad youknowriad deleted the update/pattern-categories-labels branch January 23, 2024 09:15
'not_found' => __( 'No pattern categories found.' ),
'popular_items' => __( 'Popular Pattern Categories' ),
'search_items' => __( 'Search Pattern Categories' ),
'separate_items_with_commas' => __( 'Separate pattern categories with commas' ),
Copy link
Copy Markdown
Member

@mukeshpanchal27 mukeshpanchal27 Jan 23, 2024

Choose a reason for hiding this comment

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

For "pattern categories", we use different text cases that need to be consistent.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

separate_items_with_commas is lowercase for regular tags by default, so this seems correct.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I did copy from the tags I think and adapted the labels.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For Link Category taxonomy we use 'back_to_items' => __( '← Go to Link Categories' ), and for Pattern Category we use 'back_to_items' => __( '← Go to pattern categories' ), it should be 'back_to_items' => __( '← Go to Pattern Categories' ),

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Here's a follow-up #5930

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants