Skip to content

Casimir43/enhancement/660 update prg page to match newest figma design - #668

Open
Casimir43 wants to merge 11 commits into
devfrom
casimir43/enhancement/660-update-prg-page-to-match-newest-figma-design
Open

Casimir43/enhancement/660 update prg page to match newest figma design#668
Casimir43 wants to merge 11 commits into
devfrom
casimir43/enhancement/660-update-prg-page-to-match-newest-figma-design

Conversation

@Casimir43

Copy link
Copy Markdown

📄 Pull Request Overview

Closes #660

🔧 Changes Made

Added a tab switcher to flip between Action Plan, Annual Report, and Association Rules. Also cleaned up the board member section and applied new design features.

Checklist Before Submission

  • Functionality: I have tested my code, and it works as expected. ✅
  • JSDoc: I have added or updated JSDoc comments for all relevant code. ✅
  • Debugging: No console.log() or other debugging statements are left. ✅
  • Clean Code: Removed commented-out or unnecessary code. ✅
  • Tests: Added new tests or updated existing ones for the changes made. ✅
  • Documentation: Documentation has been updated (if applicable). ✅

📝 Additional Information

Screenshot_20260721_123838
  • On mobile, tabs show icons instead of text since small text doesn't fit well
  • On mobile, the document images move below the text so the text can use the full width

@Skoivumaki Skoivumaki left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small UI fixes:

  • Text overflows when in split-view. (good rule is, if website looks bad on larger tablets, split-view is going to break): Image

  • Your mobile implementation of CustomSwitch is clean, but in terms of user experience, its often bad to use icons to represent complex words. Simply fix it by disabling CustomSwitch for mobile and loop the tabs as seen in Figma example 1 (mobile). Just remember to make it look good and not overflow: Image

Otherwise everything looks good 👍

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.

2 participants