Baskerville 2: The value of the content width is wrong

arthur791004 commented 2 years ago

Quick summary

The content width of the Baskerville 2 theme is incorrect so the width of the image is smaller than the content width and the image cannot occupy the entire column which looks weird.

Besides, the customer also mentioned this behavior only happened recently and it works well several days ago.

Related to 5427858-zd-woothemeswhich

Steps to reproduce

Switch to Baskerville 2 theme
Create a new post/page
Insert an image block
Preview the page
Check the width of the image

What you expected to happen

The width of the image should be the same as the actual content width, 756px

What actually happened

The width of the image is resized to 736px and doesn't fit the actual content width

Context

No response

Platform (Simple, Atomic, or both?)

Simple, Atomic, Self-hosted

Theme-specific issue?

Baskerville 2

Browser, operating system and other notes

No response

Reproducibility

Consistent

Severity

No response

Available workarounds?

Yes, easy to implement

Workaround details

Navigate to Appearance → Additional CSS
Scroll down to the bottom and set the "Media Width" to "756px"

github-actions[bot] commented 2 years ago

Support References

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

[ ] 5427858-zen
[ ] 5485427-zen

Robertght commented 2 years ago

📌 HOUSEKEEPING

Labels ✅
Priority ✅
Replicable on Core - No ( I can only see Baskerville on .org)

📌 SCRUBBING

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

📌 FINDINGS/SCREENSHOTS/VIDEO

I can confirm the behavior described happens.

📌 ACTIONS

Marked as Triaged for Quality Squad review
Transferred to https://github.com/Automattic/themes/

lizswafford commented 2 years ago

Related to this, (5485427-zen) there is the recommendation to change the wording Customizer > Additional CSS > Media Width to the following:

_Set a different media width for full-size images. Default media width is 736 px.

To align your media perfectly with your text columns, use a value of 756 px or greater._

ccwalburn commented 2 years ago

Adding clarification that the recommendation from 5485427-zen was not specific to Baskerville 2. It is recommended for the verbiage for any theme that has the Media Width field on the Additional CSS screen in the Customizer.

See the ticket conversation for details on the confusion the current verbiage causes.

Automattic / themes