Skip to content

Size header logos by height instead of width - #259

Open
amjadr360 wants to merge 1 commit into
mainfrom
fix/2393-header-logo-width-attr
Open

Size header logos by height instead of width#259
amjadr360 wants to merge 1 commit into
mainfrom
fix/2393-header-logo-width-attr

Conversation

@amjadr360

@amjadr360 amjadr360 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Issue: https://github.com/extendify/company-product/issues/2393

The site-logo block's width attribute makes WP core recompute the
image's intrinsic size from that width (site-logo.php:18-24), so a
landscape logo at width:40 renders 40px wide and ~10px tall. Setting
width also drops the is-default-size class, which is the only
selector the theme's existing logo clamp targets — the
max-height: 42px; width: auto rule in style.css:403. These three
headers were opting themselves out of the one rule that already
handles both square and landscape logos.

Removing the attribute puts them back on that clamp. It also makes
them agree with the Extendify API redesign preview, which renders the
logo under the same 42px max-height — the mismatch reported in
company-product#2393, where the preview showed a legible wordmark and
the built site showed a sliver.

Catalina Skyline goes from 48px to the shared 42px; Atlas Beacon and
Ceadar Peak from 40px.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant