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

The + icon on the Navigation Menu doesn't add menu items in the Blockbase themes #6162

Closed
happychait opened this issue Jul 6, 2022 · 7 comments
Labels
[Pri] High [Theme] Blockbase Triaged [Type] Bug Something isn't working User Report This issue was created following a WordPress customer report

Comments

@happychait
Copy link

happychait commented Jul 6, 2022

Quick summary

In the Site Editor, selecting the Navigation Menu block and clicking the + icon only brings up this toolbar, but we can't create new menus, or add menu items.

Captured at 06-07-2022 at 16 34 24@2x

Steps to reproduce

  1. Open the Site Editor
  2. Select the Navigation menu block in the header
  3. Click the + icon on the Navigation block to add menu items.

What you expected to happen

Add link option should be added to the Navigation block as in this screenshot:
Captured at 06-07-2022 at 16 38 51@2x

What actually happened

  1. The + icon simply pulls up the block's toolbar which can only be hidden by selecting the Navigation Menu block in the List View and hitting the Escape key.
  2. No way to add menu items.

Video of what happens:

Captured.at.06-07-2022.at.16.42.43.mp4

A few screenshots showing the issue:
Captured at 06-07-2022 at 16 00 43@2x

Captured at 06-07-2022 at 16 01 20@2x

Context

Discovered this bug while trying to help the user add the Navigation Menu from the Site Editor in 35992415-hc

Also confirmed the bug in exploratory testing.

Simple, Atomic or both?

Simple, Atomic

Theme-specific issue?

Yes, seems to be specific to the Blockbase and all its child themes.

Browser, operating system and other notes

No response

Reproducibility

Consistent

Severity

No response

Available workarounds?

Yes, easy to implement

Workaround details

We can workaround by:

  1. Choosing the Top toolbar option for the Site Editor preferences.
  2. Editing the Navigation Menu from the template or template parts editor instead of the Site Editor.
@happychait happychait added [Type] Bug Something isn't working User Report This issue was created following a WordPress customer report labels Jul 6, 2022
@jordesign
Copy link
Contributor

📌 SCRUBBING

  • Tested on Simple ✅
  • Tested on AT ✅
  • Tested on Self-hosted ✅

On Atomic and Self Hosted - the error was only present with the Gutenberg plugin activated

📌 FINDINGS/SCREENSHOTS/VIDEO

I was able to recreate this consistently when Blockbase and the Gutenberg plugin were both active

📌 ACTIONS

  • Marked as Triaged for Quality Squad review
  • Closed as could not replicate (please reopen the issue reoccurs)
  • Transferred to Themes Repo

@jordesign jordesign transferred this issue from Automattic/wp-calypso Jul 6, 2022
@getdave
Copy link
Contributor

getdave commented Jul 6, 2022

I tested this on Gutenberg Core trunk using Empty Theme theme and I wasn't able to reproduce.

That said there are a number of bugs with the popover that seem to have been introduced as a result of the upgrade to the core component's underlying library. One of these is that the block toolbar is appearing over the top of the Nav block. You can also see the Link UI disappearing off screen.

Not sure if those issues are related to what you're seeing here.

Screen.Capture.on.2022-07-06.at.10-56-09.mp4

@happychait
Copy link
Author

happychait commented Jul 6, 2022

@getdave In your video, the Top toolbar option is selected in your block editor, and hence the issue can't be reproduced. So that's another workaround as @tanjoymor helped find it originally.

Edited the issue description and included this second workaround.

@pbking
Copy link
Contributor

pbking commented Jul 7, 2022

I didn't realize this was Blockbase specific. I thought that this was the behavior of the Navigation Block. I'll look into this today; thank you for all the details.

@pbking
Copy link
Contributor

pbking commented Jul 7, 2022

I don't believe this is an issue with Blockbase per-se. It happens with Blockbase but I believe because of the layout of the header. The navigation is so close to the top that the panel hides the button. If I introduce space above then everything works as expected. When Gutenberg is de-activated the panel in question is rendered below the navigation which I expect is the correct behavior for a block in that position.

image

I observed the same behavior on Pendant which is not a Blockbase child theme but has a header with navigation a similar distance from the top of the page.

image

This seems to be properly represented in the issue that @getdave mentioned above.

@tanjoymor
Copy link

@pbking You are totally right! I just retested on Twenty Twenty-Two and if I move the Navigation block to the very top the same issues occur. As a point of interest, Simple sites (that don't have access to plugins) can't deactivate Gutenberg.

@pbking
Copy link
Contributor

pbking commented Jul 8, 2022

Going to close this considering the above. Hopefully this gets resolved via Gutenberg #41575 soon! It makes using that nav block extra hard!

@pbking pbking closed this as completed Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] High [Theme] Blockbase Triaged [Type] Bug Something isn't working User Report This issue was created following a WordPress customer report
Projects
None yet
Development

No branches or pull requests

5 participants