Skip to content

Roshini Seelamsetty : Fix Mentor Label Text Visibility (In Dark Mode) in Role Distribution Chart (Total Org Summary Dashboard)#5292

Open
roshini1212 wants to merge 1 commit into
developmentfrom
Roshini-Fix-Mentor-Label-Text-Visibility-(In-Dark-Mode)-in-Role-Distribution-Chart
Open

Roshini Seelamsetty : Fix Mentor Label Text Visibility (In Dark Mode) in Role Distribution Chart (Total Org Summary Dashboard)#5292
roshini1212 wants to merge 1 commit into
developmentfrom
Roshini-Fix-Mentor-Label-Text-Visibility-(In-Dark-Mode)-in-Role-Distribution-Chart

Conversation

@roshini1212
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-05-23 at 21 49 29

Related PRS (if any)

N/A

Main changes explained:

  • Updated PieChart.jsx to support improved dark mode label rendering.
  • Updated RoleDistributionPieChart.jsx to apply dedicated label classes for role distribution labels.
  • Updated TotalOrgSummary.module.css to:
    • add explicit dark/light label styling for role distribution labels
    • improve chart label readability in dark mode
    • remove duplicate selectors causing Stylelint failures
    • ensure chart text colors render correctly against dark backgrounds

How to test:

  1. Checkout the current branch.
  2. Run: npm install
    npm start
  3. Clear browser cache/site data if needed.
  4. Log in as an admin user.
  5. Navigate to: Dashboard -> Total Organization Summary -> Role Distribution Chart -> Enable dark mode.
  6. Verify that mentor role is visible in yellow segment.

Screenshots or videos of changes:

Screen.Recording.2026-05-23.at.21.28.21.mov

@netlify
Copy link
Copy Markdown

netlify Bot commented May 24, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 59e2f88
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a125a8784e8a60008ac726c
😎 Deploy Preview https://deploy-preview-5292--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@roshini1212 roshini1212 changed the title (PRIORITY HIGH) Tanmay : Fix Mentor Label Text Visibility (In Dark Mo… Roshini Seelamsetty : Fix Mentor Label Text Visibility (In Dark Mo… May 24, 2026
@sonarqubecloud
Copy link
Copy Markdown

@roshini1212 roshini1212 changed the title Roshini Seelamsetty : Fix Mentor Label Text Visibility (In Dark Mo… Roshini Seelamsetty : Fix Mentor Label Text Visibility (In Dark Mode) in Role Distribution Chart (Total Org Summary Dashboard) May 24, 2026
@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 May 24, 2026
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,

I reviewed and tested this PR locally in dark mode. The mentor role is now visible in the yellow segment as expected. However, there is overlap occurring in the Role Distribution chart labels/values, and the chart values are not changing when the date filters are updated.

Image

Copy link
Copy Markdown

@DeepighaJ DeepighaJ left a comment

Choose a reason for hiding this comment

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

Checked out to current branch and tested locally
The yellow appears as expected but the overlap values in yellow are still not visible clearly.
Date filters are not applied to the donut chart.

Image Image Image

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