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

Bantry Theme: Navigation Block lost padding between elements #6386

Open
ktyfuller604 opened this issue Aug 11, 2022 · 4 comments
Open

Bantry Theme: Navigation Block lost padding between elements #6386

ktyfuller604 opened this issue Aug 11, 2022 · 4 comments
Labels
Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report". [Platform] Atomic [Platform] Simple [Pri] Low Triaged [Type] Bug Something isn't working

Comments

@ktyfuller604
Copy link

Quick summary

Approximately a week ago the navigation block added to a page in the Bantry theme lost all padding between menu items.

Steps to reproduce

  1. Activate the Bantry theme
  2. Edit a Page
  3. Add a Navigation block to the page and choose a menu (classic or new, I did not see a difference)
  4. Preview or View the page

What you expected to happen

There should be spacing between each of the menu items. The screenshot below shows the site after custom CSS was added to add the spacing back.
Screen Shot on 2022-08-11 at 13-38-27

What actually happened

There is no spacing between menu items. The screenshot below shows the default behavior of the navigation block.
Screen Shot on 2022-08-11 at 13-40-27

Context

Customer report. #5457814-zd

Platform (Simple, Atomic, or both?)

Simple, Atomic

Theme-specific issue?

So far, I could reproduce on Bantry, but not other themes I tested (Pendant and Byrne).

Browser, operating system and other notes

No response

Reproducibility

Consistent

Severity

Some (< 50%)

Available workarounds?

Yes, easy to implement

Workaround details

With the following CSS:

.wp-block-navigation .wp-block-navigation-link {
	padding: 0 5px;
}
@github-actions
Copy link
Contributor

github-actions bot commented Aug 11, 2022

Support References

This comment is automatically generated. Please do not edit it.

  • 5457814-zen
  • 5508103-zen

@Robertght Robertght transferred this issue from Automattic/wp-calypso Aug 12, 2022
@Robertght
Copy link

Robertght commented Aug 12, 2022

📌 HOUSEKEEPING

  • Labels ✅
  • Priority ✅
  • Replicable on Core - No

📌 SCRUBBING

  • Tested on Simple ✅
  • Tested on AT ✅

📌 FINDINGS/SCREENSHOTS/VIDEO

  • I can confirm the no spacing part. From my investigation, it seems that the gap that inherits the gap value is "normal" on their site, but it used to be 8px before.

image

  • Not sure, but is this somewhat tied to the new GB update? cc @jeffikus

📌 ACTIONS

@jamiepalatnik
Copy link

I can reproduce this for some other themes as well, including Alves and Exford.

User report: 5508103-zd-woothemes

Suggested CSS (upgrade required) or switching to a different theme as a workaround.

@KirkwallDay
Copy link

This happens on the old Blank Canvas theme too (The non-FSE one).

  • 36818977-hc

@jeherve jeherve added Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report". and removed User Report labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report". [Platform] Atomic [Platform] Simple [Pri] Low Triaged [Type] Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants