Skip to content

327 Update Donors Page to match figma - #338

Merged
mehanana merged 3 commits into
mainfrom
327-update-donors-page
Aug 23, 2026
Merged

327 Update Donors Page to match figma#338
mehanana merged 3 commits into
mainfrom
327-update-donors-page

Conversation

@mehanana

Copy link
Copy Markdown
Contributor

ℹ️ Issue

Closes #327

📝 Description

Updated the donors page & modal to match the figma design

Briefly list the changes made to the code:

  1. Removed unecessary buttoms (sort by & filter by) and replaced with the "last donated" button
  2. Changed a column in the table to be contact name; also made the names fit within the column and not change the size of column if it was too long
  3. Added the email icon to the donors modal & made the "add donor" button disabled if the sections in the form weren't filled out

✔️ Verification

What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.

Provide screenshots of any new components, styling changes, or pages.
Overflow names:
Screenshot 2026-08-20 233757
Sorted by most recent:
Screenshot 2026-08-21 000743
Sorted by least recent:
Screenshot 2026-08-21 000749
Grayed out button when sections not filled in (and also email icon):
Screenshot 2026-08-21 000757
Button enabled when sections filled out:
Screenshot 2026-08-21 000813
Add donor button worked (test org donor added):
Screenshot 2026-08-21 003537
All Donors jest tests work:
Screenshot 2026-08-21 233959

🏕️ (Optional) Future Work / Notes

Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!
I believe the mock data was replaced in another ticket so I didn't touch that functionality, so we may run into merge conflicts when merging this and the other ticket into main.

@github-actions
github-actions Bot requested a review from Rayna-Yu August 22, 2026 03:45
github-actions Bot added a commit that referenced this pull request Aug 22, 2026

@nourshoreibah nourshoreibah left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks great!

github-actions Bot added a commit that referenced this pull request Aug 22, 2026
@mehanana
mehanana added this pull request to the merge queue Aug 23, 2026
Merged via the queue into main with commit a2b9ca9 Aug 23, 2026
19 checks passed
@mehanana
mehanana deleted the 327-update-donors-page branch August 23, 2026 01:30
github-actions Bot added a commit that referenced this pull request Aug 23, 2026
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.

Donors Page + New Donor Modal

2 participants