Open
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request implements the grant card component for the grant directory page as part of issue #308. The changes introduce a new card-based UI for displaying grants with an interactive list/detail view layout, replacing the previous table-based design.
Changes:
- Created a new
GrantCardcomponent with hover-based eligibility indicators and selection states - Refactored
GrantPageto display grants in a two-column layout with cards on the left and detailed view on the right - Moved grant refresh logic from
GrantViewtoGrantPagefor better state management - Updated styling for search bar, add button, and notification popup to align with new design
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/main-page/grants/grant-list/GrantCard.tsx | New component implementing grant cards with eligibility indicators and selection state |
| frontend/src/main-page/grants/GrantPage.tsx | Major refactor to two-column layout, integrated grant selection logic and refresh functionality |
| frontend/src/main-page/grants/grant-view/GrantView.tsx | Removed local state management and useEffect, now receives grant as prop |
| frontend/src/main-page/grants/styles/GrantButton.css | Updated add button styling to circular design |
| frontend/src/main-page/grants/styles/GrantSearch.css | Made search bar full-width with updated styling |
| frontend/src/main-page/grants/filter-bar/GrantSearch.tsx | Updated placeholder text |
| frontend/src/styles/notification.css | Adjusted notification popup position |
| frontend/src/main-page/MainPage.tsx | Reduced page padding for new layout |
| frontend/src/Login.tsx | Changed default redirect route |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
janekamata
reviewed
Feb 19, 2026
Contributor
janekamata
left a comment
There was a problem hiding this comment.
I have a few suggestions I think should be addressed before this is merged. Let me know if you have any questions! Good job overall though. Just being detailed on this since this is the most important page
janekamata
reviewed
Feb 19, 2026
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.
ℹ️ Issue
Closes #308
📝 Description
Created grant card component to be used on the grant directory page
Briefly list the changes made to the code:
GrantCard.tsxcomponent for each grant with props so grant selected is outlined in blue and the hover to view "eligible" or "ineligible"GrantCard.tsxcomponent onGrantPage.tsx, along with Jane's component for the expanded grant view so that when a grant is clicked, it displays that grant's information on the left.GrantView.tsxtoGrantPage.tsxso both components can rerender when the variables change. I also edited the UseEffect to update the entire grant array.✔️ Verification
Test Changes
If your new feature required some test to be changed or added to fit the new functionality or changes please document these changes here.
N/A
🏕️ (Optional) Future Work / Notes
Notes: