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

Change the navigation menu add block button tooltip #39602

Open
theminaldiwan opened this issue Mar 21, 2022 · 10 comments
Open

Change the navigation menu add block button tooltip #39602

theminaldiwan opened this issue Mar 21, 2022 · 10 comments
Assignees
Labels
[Block] Navigation Affects the Navigation Block Good First Issue An issue that's suitable for someone looking to contribute for the first time Needs Dev Ready for, and needs developer efforts [Type] Enhancement A suggestion for improvement.

Comments

@theminaldiwan
Copy link
Contributor

theminaldiwan commented Mar 21, 2022

What problem does this address?

While using navigation block and when we adding new menu on that we facing problem adding menu from where because menu add block button tooltip is same as other block button.

What is your proposed solution?

Need to changing add block button for navigation block menu item and 'Add block' button tooltip name to 'Add Menu Item'.
screenshot:
menu

@paaljoachim paaljoachim added the [Block] Navigation Affects the Navigation Block label Mar 30, 2022
@paaljoachim
Copy link
Contributor

paaljoachim commented Mar 30, 2022

@getdave here is a Navigation block issue to take a closer look at.

@getdave
Copy link
Contributor

getdave commented Mar 31, 2022

@theminaldiwan thank you for taking the time to raise this Issue. It's been added to the tracking issue for the block.

@theminaldiwan
Copy link
Contributor Author

Thanks @getdave and @paaljoachim for you response.

@github-actions
Copy link

Hi,
This issue has gone 180 days without any activity. This means it is time for a check-in to make sure it is still relevant. If you are still experiencing this issue with the latest versions, you can help the project by responding to confirm the problem and by providing any updated reproduction steps.
Thanks for helping out.

@github-actions github-actions bot added the Needs Testing Needs further testing to be confirmed. label Sep 28, 2022
@Thelmachido
Copy link

Hi @getdave it looks like this is on the to-do list.

Just checking if there's still a plan to move this forward.

@Thelmachido Thelmachido added [Type] Enhancement A suggestion for improvement. and removed Needs Testing Needs further testing to be confirmed. labels Sep 29, 2022
@getdave
Copy link
Contributor

getdave commented Oct 4, 2022

Seems simple enough. Low hanging fruit. Possible good first issue.

@getdave getdave added the Good First Issue An issue that's suitable for someone looking to contribute for the first time label Oct 4, 2022
@paaljoachim paaljoachim added the Needs Dev Ready for, and needs developer efforts label Oct 5, 2022
@koneweczka
Copy link

@getdave Can I take it? I think I can do this one :)

@getdave
Copy link
Contributor

getdave commented Oct 14, 2022

@getdave Can I take it? I think I can do this one :)

Please do 🙇

@koneweczka
Copy link

Ok, so I forked it and here I go :) Can you assign this issue to me?

koneweczka added a commit to koneweczka/gutenberg that referenced this issue Oct 18, 2022
@koneweczka
Copy link

So - after first look at it -I already find the right place, but have some thoughts. This navigation menu use this ButtonBlockAppender as "inner block" button. This ButtonBlockAppender is also used in e.g. adding columns. If I change this block label to "Add menu item" it also impacts e.g. columns item (or other blocks, I just checked few of them) - here's the screen.

So - right now the simplest solution can be just changing it to "Add new item" (Or maybe"Add block item"?) to differ it from "Add block". I already changed it in this commit.

BUT - I was wondering: maybe we can think about including the block name and this can looks like this "Add ${blockName} item" e.g. "Add column item", "Add nav menu item". I was trying to do this right now, but inserted value have some problem with methods for translation and converting strings, but I can try to check it again and do some more research.

What do you think? Is this small change enough? Or maybe we (I :D) can do more?

gutenberg-add-item

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 Good First Issue An issue that's suitable for someone looking to contribute for the first time Needs Dev Ready for, and needs developer efforts [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

5 participants