Skip to content

WEB-102: Move alumni highlights from About page to Team page#151

Open
ysfabdullah wants to merge 9 commits intomainfrom
web-102
Open

WEB-102: Move alumni highlights from About page to Team page#151
ysfabdullah wants to merge 9 commits intomainfrom
web-102

Conversation

@ysfabdullah
Copy link
Copy Markdown
Collaborator

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
code-the-change-yyc-site Ready Ready Preview, Comment Mar 23, 2026 4:49pm

AbiT-246
AbiT-246 previously approved these changes Feb 24, 2026
Copy link
Copy Markdown
Collaborator

@AbiT-246 AbiT-246 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@anthonyych4n anthonyych4n left a comment

Choose a reason for hiding this comment

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

Please change the alumni div to different color to not match the background, cuz it kinda just looks awkward and thats also bad design.

Other than that, looks good to me

Image

@AbiT-246
Copy link
Copy Markdown
Collaborator

AbiT-246 commented Mar 9, 2026

Looks much better. I'd say the only thing is the carousel arrows blend into the white background so they're not visible rn. The company/role labels are also less visible.

image

You might want to pull from the branch first to avoid import errors again.

@AbiT-246
Copy link
Copy Markdown
Collaborator

Looks much better now... @anthonyych4n I'm good to merge if you are

@AbiT-246 AbiT-246 self-requested a review March 23, 2026 16:49
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