feature/exec grid#69
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new “Meet the SSA Team” section to the About page by introducing reusable About-specific components for rendering executive member cards in a responsive grid, backed by a static data module.
Changes:
- Updated the About page to render a new
ExecGridsection instead of placeholder content. - Added
ExecGridto lay out exec members responsively and render anExecCardper member. - Added
execData.tsto define theExecMembertype and a hardcoded list of members/photos.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| web/src/app/about/page.tsx | Wires the new exec grid section into the About page. |
| web/src/app/about/_components/ExecGrid.tsx | Introduces the responsive grid layout for exec cards. |
| web/src/app/about/_components/ExecCard.tsx | Adds the interactive, image-based exec card (hover/tap overlay). |
| web/src/app/about/_components/execData.ts | Provides the exec member data model and hardcoded member list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…refactor touch event handlers in AboutPage
oorjagandhi
reviewed
May 12, 2026
| year: string | ||
| } | ||
|
|
||
| export const execMembers: ExecMember[] = [ |
Collaborator
There was a problem hiding this comment.
Can u change all 2024 -> 2026
| /> | ||
| <div | ||
| aria-hidden="true" | ||
| className={`absolute inset-0 bg-black/60 flex flex-col justify-end pb-6 pl-5 pr-3 pt-3 sm:pb-8 sm:pl-6 transition-opacity duration-300 ${ |
Collaborator
There was a problem hiding this comment.
Could you please up the duration to 500 instead of 300 so its a bit more gradual
…ecData years for team members Co-authored-by: Copilot <copilot@github.com>
Richman-Tan
changed the title
… removing year property Co-authored-by: Copilot <copilot@github.com>
…y; update ExecGrid key prop to use exec.id
| <div | ||
| aria-hidden="true" | ||
| className={`absolute inset-0 bg-black/60 flex flex-col justify-end pb-6 pl-5 pr-3 pt-3 sm:pb-8 sm:pl-6 transition-opacity duration-500 ${ | ||
| isActive ? 'opacity-100' : 'opacity-0 group-hover:opacity-100' |
oorjagandhi
approved these changes
May 13, 2026
Collaborator
oorjagandhi
left a comment
oorjagandhi
left a comment
There was a problem hiding this comment.
Looks good ! Nice work :D
joengy
approved these changes
May 13, 2026
Collaborator
joengy
left a comment
joengy
left a comment
There was a problem hiding this comment.
Grid looks great Richman, and the code is clean. Good job !!!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a new "Meet the SSA Team" section to the About page, displaying the executive members in a responsive grid layout. It adds the necessary components and data structure to render executive cards with names, roles, and placeholder photos, and updates the About page to include this new section.
New "Meet the SSA Team" section:
ExecCardcomponent to display individual executive member information with interactive hover/tap overlay effects, including accessibility features for keyboard navigation.ExecGridcomponent to render a responsive grid of all executive members using theExecCardcomponent.execData.tsmodule defining theExecMembertype and a hardcoded list of executive members with placeholder photos (to be replaced by real data in the future).Integration into About page:
page.tsxto import and render the newExecGridcomponent, replacing the previous placeholder content. [1] [2]#49