-
Notifications
You must be signed in to change notification settings - Fork 0
Added Second Row Component #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a new "Second Row" component to the Board page, displaying profile cards for two Co-Coordinators (Tejal Prabhu and Pascale Gibbs) with their images and titles. The component uses responsive design with flexbox layout that adapts between mobile and desktop views.
- Added BoardSecondRow component with profile cards for two board members
- Integrated the new component into the Board page
- Added missing "lint" script to package.json
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/components/Board/BoardSecondRow.tsx | New component rendering two profile cards with images, names, and titles in a responsive layout |
| src/app/board/page.tsx | Imported and rendered BoardSecondRow component on the Board page |
| package.json | Added "lint" script for Next.js linting |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
| <div className="pt-20 text-center"> | ||
| <div className="bg-mocktrial-yellow relative inset-0 mx-auto h-[300px] w-[300px] rounded-3xl"> | ||
| <div className="absolute z-10 -mt-31 mr-2"> |
Copilot
AI
Oct 22, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The second profile card uses different positioning classes ('-mt-31 mr-2') compared to the first card ('-mt-28'), creating inconsistent spacing. This inconsistency may be unintentional and could lead to alignment issues. Consider using consistent positioning values unless there's a specific design reason for the difference.
| <div className="absolute z-10 -mt-31 mr-2"> | |
| <div className="absolute z-10 -mt-28"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few more comments and your good!
Desktop:
Mobile: