Sireesha taking over Aswin fix: implemented injury-severity to the Total Construction Dashboard#4489
Open
Aswin20010 wants to merge 3 commits into
Open
Sireesha taking over Aswin fix: implemented injury-severity to the Total Construction Dashboard#4489Aswin20010 wants to merge 3 commits into
Aswin20010 wants to merge 3 commits into
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Anusha-Gali
suggested changes
Feb 12, 2026
rohanrastogi311
suggested changes
Mar 7, 2026
…odNetworkApp into aswin_add_injury_severity_card
|
DeepighaJ
requested changes
Jun 13, 2026
There was a problem hiding this comment.
- Checked out to current branch and tested PR locally.
- A new display card titled "Injury Severity by Projects" is visible and looks good.
- Filters applied as expected.
- But the calendar under dark mode needs improvement. Background goes to white when highlighted while picking a date. Please refer the last screenshot.
- The contrast of numbers in the chart can be improved so easily readable in dark mode even in mobile view.
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
This PR adds a new display card for the grouped bar graph showing Injury Severity by Projects to the Total Construction Dashboard.
Previously, the chart introduced in PR #3903 was not visible anywhere in the dashboard, resulting in low visibility of injury-related metrics.
This update ensures the injury severity data is clearly surfaced, styled consistently with the rest of the dashboard, and fully responsive with dark-mode support.
Fixes: High priority visibility issue – the injury severity dataset from PR #3903 was not displayed on the dashboard.
Related PRS (if any):
Frontend PR related to: #3903 (Injury Severity dataset introduction)
This PR exposes the UI section needed to display the data added in #3903.
No backend changes required; this PR can be tested independently from backend PRs.
Main changes explained:
Updated WeeklyProjectSummary.jsx
Updated styles in WeeklyProjectSummary.module.css
Improved dark mode compatibility
Refactored and cleaned duplicates flagged by stylelint
How to test:
Dashboard → Reports → Total Construction Dashboard
http://localhost:5173/bmdashboard/totalconstructionsummary
Screenshots or videos of changes:
PR.4489.mov
Note:
This PR does not modify backend logic.
Reviewers should focus on:
All lint/stylelint issues have been resolved, allowing successful pre-commit execution.