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

Navigation block: Fix Inaccurate description of the Show icon button setting #55429

Merged
merged 2 commits into from Nov 7, 2023

Conversation

masteradhoc
Copy link
Contributor

What?

Goal is to adjust the description of the "Show icon button" setting to be more precise.

Fixes #55163

Why?

How?

Make the description more accurate so its clear for users what to expect.

Testing Instructions

  1. Go to the site editor
  2. Add a Navigation block if there is none yet.
  3. Select the Navigation and observe the settings panel.
  4. Observe the "Show icon button' setting visually shows both the opening and closing buttons.
  5. Observe it description only mentions both the opening and closing button.

Testing Instructions for Keyboard

Screenshots or screencast

Copy link
Contributor

@alexstine alexstine left a comment

Choose a reason for hiding this comment

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

👍

@alexstine
Copy link
Contributor

CC @getdave so you are aware.

@alexstine alexstine added [Type] Copy Issues or PRs that need copy editing assistance [Package] Block library /packages/block-library [Block] Navigation Affects the Navigation Block labels Oct 18, 2023
Copy link
Contributor

@getdave getdave left a comment

Choose a reason for hiding this comment

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

Thanks for the PR (and the ping). Left a small suggestion.

Co-authored-by: Dave Smith <getdavemail@gmail.com>
@masteradhoc
Copy link
Contributor Author

@getdave no problem at all. Committed. Ready for you :)

@masteradhoc
Copy link
Contributor Author

@getdave can this be merged? :)

@getdave getdave merged commit 047e573 into WordPress:trunk Nov 7, 2023
48 checks passed
@github-actions github-actions bot added this to the Gutenberg 17.1 milestone Nov 7, 2023
@afercia
Copy link
Contributor

afercia commented Nov 7, 2023

Just a quick reminder that there is a long pending issue on Trac, also reported here on GitHub, that the verb 'toggle' should be avoided as it's hard if not impossible to translate to other languages. See:
https://core.trac.wordpress.org/ticket/34753
#42492

@getdave
Copy link
Contributor

getdave commented Nov 7, 2023

Just a quick reminder that there is a long pending issue on Trac, also reported here on GitHub, that the verb 'toggle' should be avoided as it's hard if not impossible to translate to other languages. See: https://core.trac.wordpress.org/ticket/34753 #42492

I was not aware of that. Thank you for making us aware.

What would be preferable? Or is it case-by-case?

@getdave
Copy link
Contributor

getdave commented Nov 7, 2023

Configure the visual appearance of the button that toggles the overlay menu.

Should this instead be

"Configure the visual appearance of the button that shows/hides the overlay menu."?

@afercia
Copy link
Contributor

afercia commented Nov 8, 2023

What would be preferable?

I'm not sure there has been agreement on establishing a new pattern. I'd be in favour of fixing all occurences holistically in core and the editor once a pattern has been established. For now, let's try to avoid to introduce new occurrences please and share with other the oncoing discussion.

@oandregal oandregal changed the title Navigation: Fix Inaccurate description of the Show icon button setting Navigation block: Fix Inaccurate description of the Show icon button setting Nov 16, 2023
@oandregal oandregal added the [Type] Enhancement A suggestion for improvement. label Nov 16, 2023
@masteradhoc masteradhoc deleted the patch-1 branch November 16, 2023 18:45
@getdave getdave removed the [Type] Enhancement A suggestion for improvement. label Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Package] Block library /packages/block-library [Type] Copy Issues or PRs that need copy editing assistance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigation: Inaccurate description of the Show icon button setting
5 participants