Bantry Theme: Navigation Block lost padding between elements

ktyfuller604 commented 2 years ago

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

Activate the Bantry theme
Edit a Page
Add a Navigation block to the page and choose a menu (classic or new, I did not see a difference)
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.

What actually happened

There is no spacing between menu items. The screenshot below shows the default behavior of the navigation block.

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[bot] commented 2 years ago

Support References

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

[ ] 5457814-zen
[ ] 5508103-zen

Robertght commented 2 years ago

📌 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.

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

📌 ACTIONS

Marked as Themes
Transferred to XYZ external repo: https://github.com/Automattic/themes/

jamiepalatnik commented 2 years ago

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 commented 2 years ago

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

[ ] 36818977-hc

Automattic / themes