Skip to content

Fix steering-committee card misalignment (drop 1px border) (#808)#816

Merged
richarddushime merged 1 commit into
mainfrom
fix-808-card-border
Jun 25, 2026
Merged

Fix steering-committee card misalignment (drop 1px border) (#808)#816
richarddushime merged 1 commit into
mainfrom
fix-808-card-border

Conversation

@LukasWallrich

Copy link
Copy Markdown
Contributor

Follow-up to #815. On the live page the member photo cards rendered slightly shorter and lower than the team title cards, with a gap above each photo, and the colored team header bars did not line up into one continuous row.

Root cause

.sc-card (member cards) had border: 1px solid transparent; .sc-title-card had border: 0. With box-sizing: border-box both cards measure the same outer size (so getBoundingClientRect reports them as aligned), but their absolutely-positioned children — .sc-card-photo, .sc-card-img, .sc-cbar — are painted from the padding box, 1px inside the border. So every photo and header bar on a member card sat 1px down and 2px smaller than on the title cards. On Retina that's 2px, and against the full-bleed title cards it reads as a clear height gap and a broken header line.

Fix

Remove the border from .sc-card (one line). The padding box now equals the border box, so photos fill flush to the top and the header bars align continuously across each team.

Verified in a local Hugo build (real academic theme) at 1× and 2×: title cards, photo cards, and all header bars share the same top edge; photos fill to the card top.

🤖 Generated with Claude Code

Member cards (.sc-card) had `border: 1px solid transparent` while title
cards (.sc-title-card) had `border: 0`. With `box-sizing: border-box`
both cards measure the same outer size — so getBoundingClientRect reports
them aligned — but absolutely-positioned children (.sc-card-photo,
.sc-card-img, .sc-cbar) are painted from the *padding* box, i.e. 1px
inside the border. So on photo cards the image and the team header bar
sat 1px lower and smaller than on title cards: photos looked shorter with
a gap above them, and the connecting header bars didn't line up into one
continuous row.

Removing the border makes the padding box equal the border box, so photos
fill flush to the card top and the header bars align across each team.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@LukasWallrich LukasWallrich requested a review from a team as a code owner June 25, 2026 10:07
@github-actions

Copy link
Copy Markdown
Contributor

👍 All image files/references (if any) are in webp format, in line with our policy.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Spell Check Passed

No spell-checkable files were changed in this PR.

@LukasWallrich

Copy link
Copy Markdown
Contributor Author

Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-06-25 10:12:12 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

@richarddushime richarddushime left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@richarddushime richarddushime merged commit 3387858 into main Jun 25, 2026
5 checks passed
@richarddushime richarddushime deleted the fix-808-card-border branch June 25, 2026 12: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.

2 participants