Skip to content

Mani shashank fix: match no-show insights tab and percentage styling to Figma#5023

Merged
one-community merged 4 commits intodevelopmentfrom
fix/no-show-insights-ui-styling
Apr 24, 2026
Merged

Mani shashank fix: match no-show insights tab and percentage styling to Figma#5023
one-community merged 4 commits intodevelopmentfrom
fix/no-show-insights-ui-styling

Conversation

@shashankm9
Copy link
Copy Markdown
Contributor

@shashankm9 shashankm9 commented Mar 20, 2026

Description

image image Screenshot 2026-03-19 204335

Figma:

image

Related PRS (if any):

Main changes explained:

  1. Percentage color fix
  • Renamed .insightPercentage → .insightsPercentage (was a class name mismatch, so no styles applied at all)
  • Color changed from neutral dark to red — matching the tracking table's trackingRateRed for consistency
  • Removed font-weight: bold
  1. Active tab — pill/segmented control style
  • Container (.insightsTabs): replaced bordered rectangle with a gray (#f0f0f0) rounded pill background (border-radius: 8px), no border
  • Inactive tabs: transparent background, dark gray text
  • Active tab: white background with subtle shadow — visually "lifted" within the gray container
  1. Dark mode active tab
  • Container: #334155 (visible slate, not near-black)
  • Active tab: near-white #e2e8f0 background with dark #1e293b text — strong light-on-dark contrast
  • Inactive tabs: muted #94a3b8 text
  1. Stylelint fixes (to pass the pre-commit hook)
  • Converted all rgba() → rgb(a / b%) modern notation
  • Shortened #ffffff → #fff
  • Expanded single-line declaration blocks
  • Fixed selector ordering for no-descending-specificity
  • Replaced deprecated page-break-* with break-*

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to dashboard→ http://localhost:5173/communityportal/reports/participation
  6. verify function percentage is in right color
  7. verify active tab style matches with the figma

Screenshots or videos of changes:

Recording.2026-03-19.211909.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 20, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit dd2c813
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69ead12edf09d30008f76bde
😎 Deploy Preview https://deploy-preview-5023--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 12, 2026
Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Mani,

Well done with this implementation.

Image

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Shashank,

I tested this PR locally and no-show insights tab and percentage style correctly matches with Figma Prototype.

Image Image

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 0cf46af into development Apr 24, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants