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: make limitations around which blocks can be added clearer #30159

Closed
annezazu opened this issue Mar 23, 2021 · 7 comments
Closed
Labels
[Block] Navigation Affects the Navigation Block [Type] Enhancement A suggestion for improvement.

Comments

@annezazu
Copy link
Contributor

What problem does this address?

Right now, the navigation block has inherent limits to what can be added to it which makes sense. However, the current flow is a bit confusing as it nudges a user to browse all blocks and then doesn't show any indication that only certain blocks can be used:

Navigation.Block.limitations.mov

What is your proposed solution?

Either the browse all option needs to be removed for now or there needs to be some sort of messaging that says only specific blocks can be added.

This is somewhat related to this other issue around adding some introductory text to better guide a user: #24007

@annezazu annezazu added [Type] Enhancement A suggestion for improvement. [Block] Navigation Affects the Navigation Block [Feature] Full Site Editing labels Mar 23, 2021
@talldan
Copy link
Contributor

talldan commented Mar 24, 2021

Possibly Browse All shouldn't be shown when the quick inserter already shows all the available options (though it should take into account patterns and reusable blocks).

Just also wanted to mention there's a bug in play here (#30117). Usually there would be more options available, but the Link block variations aren't working in the screen recording.

@jasmussen
Copy link
Contributor

Possibly Browse All shouldn't be shown when the quick inserter already shows all the available options (though it should take into account patterns and reusable blocks).

That does seem the simplest solution, though an alternative could be a different label. I wonder if just "Browse" would work, or actually make it worse? The point is, it opens the inserter on the left, showing you the available blocks in a different interface.

@jasmussen
Copy link
Contributor

I just created #33278 to expand the range of blocks allowed in navigation.

@jasmussen
Copy link
Contributor

Coming back to this ticket, a few things feel like they will address this ticket together:

  • A Lighter Navigation Block Experience #34041 defaults to just inserting page links, which will make it vastly simpler to make basic menus.
  • The expanded range of blocks allowed (including site logo as well as title) means more items you might intuit to be available, are.
  • The introductory text you suggested yourself and mocked up here should surface the last bit of context.

@annezazu
Copy link
Contributor Author

Agreed completely. Thanks for looping back. Happy to close this out if you think it's no longer relevant?

@jasmussen
Copy link
Contributor

Oh I think it's relevant enough! Just making a note to self that once the three items above are addressed, this one very probably is too.

@annezazu
Copy link
Contributor Author

The UI has come so far since I opened this issue and I find it to be MUCH more intuitive. I'm going to close this out as a result. I also believe the wider work around making blocks more context aware will inherently continue to trickle down into this experience.

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 [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants