Skip to content

Conversation

@naiduajaykumar
Copy link
Contributor

@naiduajaykumar naiduajaykumar commented Dec 24, 2025

Description

image image

Related PRS (if any):

This frontend PR is not related to any backend PR.

Main changes explained:

  • Fixed the theme issues
  • Made cards responsiveness across desktop and tablet views

How to test:

  1. check into current branch
  2. do yarn install and run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to dashboard→ Reports→ Total Construction Summary→ Project Status
  6. verify them

Screenshots or videos of changes:

PR-4619-1.mp4

@netlify
Copy link

netlify bot commented Dec 24, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 9552367
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/697148166843ea0009d159b6
😎 Deploy Preview https://deploy-preview-4619--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.

@JuliaHa0902
Copy link
Contributor

I tested, and the responsiveness looks good in the Project Status page. However, the text is white and has very low contrast with the background, making it hard to read in dark mode. You can see the image below
Screenshot 2025-12-26 153311
Does this PR of yours include other charts in the page? If yes, there a a few other problems with dark mode with other charts:
Screenshot 2025-12-26 153803
Screenshot 2025-12-26 153811
Screenshot 2025-12-26 153826

@naiduajaykumar
Copy link
Contributor Author

I tested, and the responsiveness looks good in the Project Status page. However, the text is white and has very low contrast with the background, making it hard to read in dark mode. You can see the image below Screenshot 2025-12-26 153311 Does this PR of yours include other charts in the page? If yes, there a a few other problems with dark mode with other charts: Screenshot 2025-12-26 153803 Screenshot 2025-12-26 153811 Screenshot 2025-12-26 153826

I had just modified the change you had asked for, can also please review this again. Also the other charts which you had mentioned are not mine. Thank you

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants