Sudheesh - Fix: chart visibility issues in Dark Mode- Most Susceptible tools Chart#4754
Merged
one-community merged 4 commits intodevelopmentfrom Mar 2, 2026
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
JuliaHa0902
previously approved these changes
Jan 30, 2026
Anusha-Gali
previously approved these changes
Feb 1, 2026
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Sudheesh,
I have reviewed your PR locally and was able to successfully test in in both the modes. I would however like to point out two issues which do appear out of scope of this PR.
- The link is not protected.
- When there is no data in chart, a partial ghost chart row appears. Do let me know if you fix it.
4754.mp4
sayali-2308
previously approved these changes
Feb 2, 2026
Contributor
sayali-2308
left a comment
There was a problem hiding this comment.
- Branch:
Sudheesh-chart-visibility-issue-mostsusceptible-tools-chart - Backend:
developmentbranch - Tested on: Windows, Chrome
- Date: January 31, 2026
Dark Mode Testing:
- Navigated to
/mostsusceptibletoolschart - Chart title "Tools Most Susceptible to Breakdown" - white text, clearly visible
- Project dropdown - "All Projects" visible with light text
- Date Range display - readable with proper contrast
- Start Date and End Date input fields - light text on dark inputs, clearly readable
- Apply button - blue, clearly visible
- Horizontal bar chart - blue bars display correctly
- Y-axis tool labels - all visible in white text (Wrench, Saw, Pliers, Screwdriver)
- Percentage values on bars - white text, clearly readable (53%, 52%, 52%, 32%)
- X-axis percentage labels - white text (0%, 25%, 50%, 75%, 100%)
- Gridlines - subtle dark lines, visible and don't overpower the chart
- Background - dark blue-gray with excellent contrast
Light Mode Testing:
- Switched to light mode
- Chart title - dark text on light background, clearly visible
- Project dropdown - light background with dark text
- Date Range - readable
- Chart displays correctly with 6 tools (Drill 65%, Hammer 56%, Wrench 53%, Saw 52%, Pliers 52%, Screwdriver 32%)
- Y-axis labels - dark text, clearly readable
- Percentage values - visible on bars
- X-axis labels - dark text, clearly readable
- Gridlines - light gray, visible with proper contrast
- Background - light/white with excellent contrast
Functionality:
- Date range picker works correctly
- Project dropdown functional
- Chart renders properly in both themes
- Data displays accurately
- No console errors observed
naiduajaykumar
previously approved these changes
Feb 5, 2026
saitejakaasoju
previously approved these changes
Feb 6, 2026
e092800
|
HemanthNidamanuru
approved these changes
Feb 27, 2026
Anusha-Gali
approved these changes
Feb 27, 2026
Member
|
Thank you all, merging! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

























Description
Related PRS (if any):
To test this PR you need to checkout the Latest Development PR.
…
Main changes explained:
…
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes:
Note:
Include the information the reviewers need to know.