Skip to content

Conversation

@uhakruthi1
Copy link
Contributor

@uhakruthi1 uhakruthi1 commented Nov 23, 2025

Fix: Longest Open Issues — Dark Mode Styling, Layout Spacing, and Responsive Chart Improvements

Screenshot 2025-11-23 at 5 16 53 PM

How to Test
related backend pr: PR#1988
checkout this backend pr

  1. Checkout branch
    git fetch
    git checkout feature/longest-open-issues-fixes
    npm install
    npm start donot give this please

  2. Navigate to the Chart
    Go to:
    /bmdashboard/totalconstructionsummary

  3. Verify Layout Fixes
    • Chart stretches to full width
    • Black background for the entire component is 100% width
    • No overlapping labels
    • Left/right margins look clean and consistent

  4. Verify Dark Mode
    • Background is fully dark (no white patches)
    • Tooltip uses dark styling
    • Dropdown uses correct dark-mode colors
    • Bars, labels, and grid lines are readable

Screenshots:

Screenshot 2026-01-06 at 12 30 47 AM Screenshot 2026-01-06 at 12 30 58 AM

@netlify
Copy link

netlify bot commented Nov 23, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 29a8280
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6964190a3d0c840008fe81e7
😎 Deploy Preview https://deploy-preview-4441--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

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@uhakruthi1 uhakruthi1 changed the title longest-open-issues uha-longest-open-issues Nov 23, 2025
@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 Jan 5, 2026
Copy link

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Uha,

I tried reviewing your PR locally though many requirements are satisfied i am unable to see the color coded bars as mentioned in the requirement and your screenshot.
Screenshot 2026-01-06 at 5 16 44 PM
Screenshot 2026-01-06 at 5 19 00 PM

@uhakruthi1 uhakruthi1 requested a review from Anusha-Gali January 6, 2026 22:36
@uhakruthi1
Copy link
Contributor Author

Hi Uha,

I tried reviewing your PR locally though many requirements are satisfied i am unable to see the color coded bars as mentioned in the requirement and your screenshot. Screenshot 2026-01-06 at 5 16 44 PM Screenshot 2026-01-06 at 5 19 00 PM

The frontend Pr is working fine for locally, just tested.

Anusha-Gali
Anusha-Gali previously approved these changes Jan 8, 2026
Copy link

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Uha,

Thanks for updating the code.
Screenshot 2026-01-07 at 10 11 16 PM
Screenshot 2026-01-07 at 10 12 23 PM

@uhakruthi1 uhakruthi1 changed the title uha-longest-open-issues uha longest open issues Jan 11, 2026
@one-community one-community changed the title uha longest open issues uha - P2 - fix issues for horizontal bar graph showing longest open issues Jan 11, 2026
@sonarqubecloud
Copy link

@uhakruthi1 uhakruthi1 changed the title uha - P2 - fix issues for horizontal bar graph showing longest open issues uha fix issues for horizontal bar graph showing longest open issues Jan 11, 2026
@one-community
Copy link
Member

Thank you all, merging!

@one-community one-community merged commit b7666a6 into development Jan 12, 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