Skip to content

Fix mobile sponsor logo sizing and spacing#116

Merged
tanjz12 merged 3 commits into
mainfrom
devin/1782854044-sponsor-mobile-sizing
Jun 30, 2026
Merged

Fix mobile sponsor logo sizing and spacing#116
tanjz12 merged 3 commits into
mainfrom
devin/1782854044-sponsor-mobile-sizing

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

A previous attempt to make Gold/Blue sponsor logos smaller and more condensed on mobile had no visible effect. Two root causes:

  1. Whitespace wasn't controlled by gap. The mobile grids used grid-cols-N + justify-items-center, which creates equal-width columns (each 1/N of the container) with the logo centered inside. The visible space between logos came from the wide columns, so shrinking gap-x did almost nothing.
  2. Sizing didn't take. Logos render at their natural size when it's below the max-w/max-h cap, so reducing max-h-12 → max-h-10 was barely noticeable. Separately, Diamond used max-h-18, which isn't a real Tailwind class (the default scale skips 18) so it never compiled.

Fixes (all in content/_index.md, mobile only — desktop reverts via sm: and is unchanged):

  • Columns are now content-sized so the gap is the actual spacing:
    grid-cols-[repeat(2,auto)] / grid-cols-[repeat(3,auto)] + justify-center.
  • Logos use a fixed box + object-contain so they actually scale to the requested size (up for Diamond, down for Gold/Blue), e.g.
    • Diamond: w-32 h-16 object-contain (larger)
    • Gold: w-28 h-10 object-contain (smaller) + tighter gap-x-4
    • Blue: w-20 h-10 object-contain (smaller) + tighter gap-x-3
  • Gold's lone third logo (ContestDojo) is centered on its second row via [&>a:last-child]:col-span-2 [&>a:last-child]:justify-self-center.

Verified at ~390px (mobile) and full width (desktop).

Link to Devin session: https://app.devin.ai/sessions/2c62d21ae9ab454ca4fe0b9e0d559825

…ller condensed gold/blue

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@tanjz12 tanjz12 merged commit 63ce061 into main Jun 30, 2026
2 checks passed
@tanjz12 tanjz12 deleted the devin/1782854044-sponsor-mobile-sizing branch June 30, 2026 21:44
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