-
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
Block Navigation: Change the visible labels for "Block navigation" to "List view". #23796
Conversation
Size Change: +4 B (0%) Total Size: 1.14 MB
ℹ️ View Unchanged
|
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.
I'm not sure what the convention is for renaming components, but just renaming the user-facing strings is already a big improvement, and renaming the components could happen in another PR, so approving.
@@ -34,7 +34,7 @@ function BlockNavigation( { | |||
return ( | |||
<div className="block-editor-block-navigation__container"> | |||
<p className="block-editor-block-navigation__label"> | |||
{ __( 'Block navigation' ) } | |||
{ __( 'List view' ) } |
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.
Personally, I would prefer "Tree view", but this is already a big improvement over the current name.
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.
To me, tree's are better in the forest. Kidding aside, I'm not sure most people know the concept of a "tree" view — it feels a little "inside baseball;" Jargon that is mostly unknown.
A list is a really simple concept, and I suspect more universally understandable.
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.
Fair point! 👍
Fixes #22814
This PR changes the label "Block navigation" to "List view":
Unclear to me if we should rename the component in general.