[Bug]: Menu overlays don't have sufficient padding

[dsas]

Quick summary

Some themes use unit-less values for styles.spacing.padding in theme.json which means the overlay menu is displayed without padding

Unit-less values	United values

Steps to reproduce

1. Switch to a site with one of the potentially affected themes
2. Change the viewport to mobile
3. Click the menu button

What you expected to happen

The overlay menu should appear with spacing so that menu items are not butted up against the sides of the screen

What actually happened

The overlay menu hugs the side of the screen

Browser

No response

Context

First discovered in D128657-code for these themes

Platform (Simple, Atomic, or both?)

No response

Themes to check

• premium/annalee
• premium/arkin never launched
• premium/carnation
• premium/gourmetto
• premium/outland
• premium/peirao
• premium/tenku
• premium/zaino
• pub/aldente relaunched under diff name
• pub/artly
• pub/bibimbap
• pub/bitacora
• pub/bute
• pub/exmoor
• pub/fewer
• pub/fotograma
• pub/freddie
• pub/jinjang
• pub/kigen
• pub/lettre (maybe, it uses empty strings rather than zero?)
• pub/loic
• pub/luce
• pub/luminance
• pub/montagna relaunched under diff name
• pub/optimismo
• pub/poesis
• pub/storia
• pub/strand
• pub/tenaz relaunched under diff name
• pub/tomoni
• pub/trellick
• pub/tronar
• pub/tu

Other notes

I generated the list of themes to check by running this command on my sandbox in ~/public_html/wp-content/themes:

for FILE in `ls */*/theme.json`; do echo $FILE; jq .styles.spacing.padding $FILE; done | grep -v px | grep -v preset | grep -v null

and then doing some manual tidy up.

I've publicised the issue so that hopefully we avoid this in future. See pNEWy-hUC-p2

Reproducibility

Consistent

Severity

All

Available workarounds?

No but the platform is still usable

Workaround details

You can probably add some custom CSS...

Themes to fix

Beta Give feedback

1. premium/annalee
2. premium/carnation
3. premium/gourmetto
4. premium/outland
5. premium/peirao
6. pub/artly
7. pub/bibimbap
8. pub/bute
9. pub/exmoor
10. pub/fewer
11. pub/fotograma
12. pub/freddie
13. pub/jinjang
14. pub/kigen
15. pub/lettre
16. pub/loic
17. pub/luce
18. pub/poesis
19. pub/storia
20. pub/tomoni
21. pub/trellick
22. pub/tronar
23. pub/tu

[maciejpilarski]

📌 REPRODUCTION RESULTS
– Tested on Atomic – Replicated

📌 FINDINGS/SCREENSHOTS/VIDEO
I could not find the following themes in the https://wordpress.com/themes/ repo (not tested):

• arkin
• aldente
• montagna
• optismo
• strand
• tenaz

I was not able to reproduce the issue in (it might be because the menu items are centered):

• tenku
• zaino
• bitacora
• luminance

📌 ACTIONS

• Triaged

[dsas]

Thank you very much @maciejpilarski for testing all of those themes.

Of the ones you could not find:

• Arkin was never launched
• Aldente, montagna & tenaz were relaunched under different names (see pNEWy-hp0-p2)
• Strand is here: https://wordpress.com/theme/strand/
• Optimismo is here (I had a typo): https://wordpress.com/theme/optimismo/

[iamtakashi]

Thanks, @dsas, for starting this issue.

How should we tackle this? Should each designer own and fix their themes, and could you help with the reviewing, merging, and deployment?

cc: @beafialho @henriqueiamarino

[dsas]

Whatever works best for you @iamtakashi, I'm happy to help with reviewing etc, just let me know.

[philnick206]

Reported in 8016038-zd-a8c

[github-actions]

Support References

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

• 8016038-zen
• 8801728-zen

[jartes]

Another one with Artly 8801728-zd-a8c