Skip to content

Polish people cards - #8

Merged
meow-wwww merged 3 commits into
datavisards:mainfrom
thinhnguyentruong:people-page-updates
Oct 4, 2026
Merged

meow-wwww merged 3 commits into
datavisards:mainfrom
thinhnguyentruong:people-page-updates

Conversation

@thinhnguyentruong

Copy link
Copy Markdown
Contributor

Changes

  • Avatars are full circles with a subtle ring, and slightly larger
  • Cards are wider so names fit on one line
  • Program and period on separate lines; open periods show as "2025–present"
  • Co-supervisor line reads "Co-advised by …" and keeps the name together
  • Updated Theo's name, photos (front and hover), and website

Before

Before

After

After

@thinhnguyentruong

Copy link
Copy Markdown
Contributor Author

@arpitnarechania @meow-wwww could one of you take a look when you get a chance? Thanks!

@meow-wwww
meow-wwww merged commit 8dd5c3d into datavisards:main Oct 4, 2026
2 checks passed
@meow-wwww

Copy link
Copy Markdown
Contributor

Merged and deployed. Thanks! 🫡

@arpitnarechania

Copy link
Copy Markdown
Contributor

Somehow still shows up as below on Safari + In cognito.

image

@thinhnguyentruong

Copy link
Copy Markdown
Contributor Author

@arpitnarechania Found it — Safari doesn't honor aspect-ratio on the avatar box inside the flex card, so it stretches vertically. Switched to a padding-based square and verified in WebKit. Fix in #9.

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.

3 participants