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

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

Conversation

youknowriad
Copy link
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

@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
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
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
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
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
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
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