givewp icon indicating copy to clipboard operation
givewp copied to clipboard
verified publisher icon impress-org

Refactor: improve Donor-Overview component architecture

Open JoshuaHungDinh opened this issue 6 months ago • 1 comments

Description

This PR refactors the Donor-Overview component to improve its architecture and maintainability. The changes focus on enhancing the component structure and layout by creating specific container components to house each panels logic.

Changes

  • Enhanced modular component structure and layout architecture
  • Cleaned up unnecessary comments
  • Added unreleased tag documentation

Testing Instructions

Please verify:

  • Donor overview page loads correctly
  • All donor information displays properly
  • No visual regressions in the donor details overview

Pre-review Checklist

  • [x] Acceptance criteria satisfied and marked in related issue
  • [x] Relevant @unreleased tags included in DocBlocks
  • [ ] Includes unit tests
  • [ ] Reviewed by the designer (if follows a design)
  • [x] Self Review of code and UX completed

JoshuaHungDinh avatar Jun 11 '25 23:06 JoshuaHungDinh

Sounds good! I thought the same thing after I peaked at your PR yesterday! I'll also double check. 😄

JoshuaHungDinh avatar Jun 18 '25 16:06 JoshuaHungDinh

@pauloiankoski So I moved the styles for the DonorSummary & DonorTransactions into their own component sheet. Referenced your main stylesheet here https://github.com/impress-org/givewp/pull/7981/files to add the missing changes. 8a9260d997c99e789e974dfc612cd90158cd4c56 9cda5fddef0173d0783a742869b6cf39187a0426 151958e1e644837702a58d3efaadd89fc78aca72 8daba41221fbe0c20b9bbb0ae893d1ad1ca32057 3885089007def10027a43f88605818e345b85113

JoshuaHungDinh avatar Jun 27 '25 19:06 JoshuaHungDinh