Skip to content

Network logo sizes: equal area instead of equal height - #84

Merged
lolman420 merged 1 commit into
mainfrom
fix/network-logo-sizes
Oct 28, 2026
Merged

lolman420 merged 1 commit into
mainfrom
fix/network-logo-sizes

Conversation

@lolman420

Copy link
Copy Markdown
Contributor

Closes #80

Problem

Network logos were capped by a shared max-height (28 px in the footer, 44 px in the partner grid). Wide wordmarks like Visma and Nokia filled their cell while the nearly square UPM mark ended up a fraction of their size.

Fix

New partial partner-logo.html gives each logo the area of a fixed square (34 px side in the footer, 48 px in the grid) and derives width/height from the logo's aspect ratio. Wide logos come out long and low, tall ones narrow and high, so they read as the same visual weight. CSS no longer caps height; max-width: 100% still shrinks a logo to fit its cell.

Footer at 375 px wide, before → after: UPM ~23×28 → 31×38, Visma full cell width → 72×16.

Footer logos are now loading="lazy", as the grid's already were.

Tested

Built with Hugo and checked the footer at mobile width and the partner grid on /en/yrityssuhteet/ at desktop width.

🤖 Generated with Claude Code

A shared max-height let wide wordmarks like Visma fill their cell while the
nearly square UPM mark shrank to a fraction of its size. Each logo now gets
the area of a fixed square, so wide logos come out long and low and tall
ones narrow and high, and they read as the same weight. Used by both the
footer and the partner grid.

Closes #80

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Esikatselu on poistettu, koska tämä pull request on suljettu.

@lolman420

Copy link
Copy Markdown
Contributor Author

@nlinnanen huomasiks tän töissä xdd

@lolman420
lolman420 merged commit bb5b334 into main Oct 28, 2026
5 checks passed
@lolman420
lolman420 deleted the fix/network-logo-sizes branch October 28, 2026 16:45
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.

Network logo sizes

1 participant