Skip to content

feat: added basic members page and MUI styling - #12

Open
pgovindu08 wants to merge 5 commits into
mainfrom
pranav-govindu-member-list
Open

feat: added basic members page and MUI styling#12
pgovindu08 wants to merge 5 commits into
mainfrom
pranav-govindu-member-list

Conversation

@pgovindu08

@pgovindu08 pgovindu08 commented Mar 29, 2026

Copy link
Copy Markdown
Collaborator

Description

Added a members list with placeholder data that shows each member's bio, LinkedIn, and GitHub contributions in a detail panel. The page also has a search bar to filter members by name, major, role, or graduation year. Note: the members list currently uses mock data as a placeholder until a real data source is connected.

Relevant Issues

How to Test

Miscellaneous Notes

  • The "Filters" button is rendered in a disabled state until filter behavior is implemented.
  • Replaced maxWidth: 'lg' in sx with MUI's Container component (maxWidth="lg") for correct theme-aware breakpoint handling.
  • Fixed "Github" label capitalization to "GitHub" in the member detail panel.

Copilot AI review requested due to automatic review settings March 29, 2026 17:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a client-side /members page implemented with Material UI components, including a searchable members table and a detail panel showing bios and external links.

Changes:

  • Implemented MembersPage with search filtering, selection, and a two-pane layout.
  • Added MemberTable and MemberDetail UI components.
  • Introduced placeholder member data and GitHub contributions chart rendering.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/app/members/page.tsx Outdated
Comment thread src/app/members/page.tsx
Comment thread src/app/members/page.tsx Outdated
Comment thread src/app/members/page.tsx Outdated
Comment thread src/app/members/page.tsx
Comment thread src/app/members/page.tsx Outdated
pgovindu08 and others added 2 commits March 30, 2026 11:34
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@pgovindu08

Copy link
Copy Markdown
Collaborator Author

@copilot apply changes based on the comments in this thread

pgovindu08 and others added 2 commits April 2, 2026 00:21
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI commented Apr 2, 2026

Copy link
Copy Markdown

@copilot apply changes based on the comments in this thread

Applied all the unresolved review changes in commit a43bae0:

  1. Filters button — added disabled prop so it no longer renders as a clickable but non-functional control.
  2. maxWidth: 'lg' in sx — replaced the outer Box with MUI's Container component using maxWidth="lg", which properly resolves the breakpoint via the MUI theme.
  3. "GitHub" capitalization — was already corrected in a prior commit via rebase.

The mock data concern (lines 36–39) is noted — the data is intentional placeholder content while the real data source is wired up.

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