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: prompt to add link/select post only when selecting individual menu item #50479

Closed
annezazu opened this issue May 9, 2023 · 3 comments
Labels
[Block] Navigation Affects the Navigation Block Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.

Comments

@annezazu
Copy link
Contributor

annezazu commented May 9, 2023

What problem does this address?

This came up as part of the FSE Outreach Program's Front Page Fun call for testing:

one example is when you’re editing the navigation block and want to add a new item. Suppose you don’t choose content for your item (for any reason). If you click outside the sidebar to exit, do something else, and then click again in any part of the navigation block afterwards — not in the navigation item you were working with — you are presented with a popup to select the content for the previously created navigation item (GIF). The previous context is remembered for me, but I was not expecting that. I wasn’t even thinking about that item anymore. It took some time for me to understand what the UI was trying to nudge me to select the content. I think this should either be removed or made more obvious. That popup should only appear if I click specifically on the item I was editing before, not in other areas of the navigation block itself.

Here's a video:

adding.link.mov

What is your proposed solution?

Only prompt someone to add a link or select a block when the individual item is selected rather than when the entire navigation block is selected. Consider adding in more context as well rather than just opening the inserter experience.

@annezazu annezazu added [Type] Enhancement A suggestion for improvement. Needs Design Feedback Needs general design feedback. [Block] Navigation Affects the Navigation Block labels May 9, 2023
@annezazu
Copy link
Contributor Author

annezazu commented May 9, 2023

@getdave this might be worth considering as part of larger link control work.

@draganescu
Copy link
Contributor

I cannot reproduce this on trunk anymore:

navigation-popover-correct.mp4

@getdave
Copy link
Contributor

getdave commented May 22, 2023

This should also never be a problem again due to fix in #50774 and accompanying e2e test (regression) coverage I added.

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 Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants