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: Move the "add submenu" item and add an icon #46072

Closed
scruffian opened this issue Nov 25, 2022 · 5 comments · Fixed by #46884
Closed

Navigation: Move the "add submenu" item and add an icon #46072

scruffian opened this issue Nov 25, 2022 · 5 comments · Fixed by #46884
Assignees
Labels
[Block] Navigation Affects the Navigation Block

Comments

@scruffian
Copy link
Contributor

What problem does this address?

The "add a submenu" item in the navigation link block is hard to find.

What is your proposed solution?

In the navigation link block, the “add submenu item” should be at the top of the ellipsis menu options with a divider below. We should also use the submenu icon to reinforce the connection.

@scruffian scruffian added the [Block] Navigation Affects the Navigation Block label Nov 25, 2022
@Humanify-nl
Copy link

Added to this; when the site has a black background color, the plus signs are black too. Making it even harder.

@getdave
Copy link
Contributor

getdave commented Dec 7, 2022

Added to this; when the site has a black background color, the plus signs are black too. Making it even harder.

@Humanify-nl I think that's the appender in the block canvas though right? This issue pertains to the experimental offcanvas editor for the Navigation block.

If I got that wrong feel free to provide a screenshot 🙇

@Humanify-nl
Copy link

@getdave I believe you are correct sir. It is the little plus inside the submenu I am referring to.

@draganescu draganescu self-assigned this Dec 8, 2022
@draganescu
Copy link
Contributor

I am either not looking at this correctly or customizing the more menu for one case seems very difficult. That menu is meant to hold:

  • generic options
  • available for all blocks
  • registered once

... and this shows in the API:

  • items added via plugins are added after default options
  • there is no way for a block to decide to opt out of these options
  • there is no way to decide which is 1st or 2nd

I am unsure how to approach this for the off canvas navigation editor component, as even if it is open to implement some workarounds, I can't see them (again, I may be looking at this the wrong way).

Do we really need the normal block more menu? Can't we build our own for this case?

If I were to build a smaller options menu for the experiment, some things would be more discoverable for the experiment validation. Deciding to use this special more menu could be a prop of the ListView, so that won't block a future merge of the off canvas navigation editor into the ListView.

@draganescu
Copy link
Contributor

I will explore the idea of building a custom menu for the navigation block. This menu will be passed via prop and displayed instead of the normal one.

@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Jan 4, 2023
@priethor priethor removed the [Status] In Progress Tracking issues with work in progress label Jan 11, 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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants