Skip to content

Vinay K Project Risk Profile#4771

Open
Vinay944924 wants to merge 3 commits intodevelopmentfrom
vinay_k_project_risk
Open

Vinay K Project Risk Profile#4771
Vinay944924 wants to merge 3 commits intodevelopmentfrom
vinay_k_project_risk

Conversation

@Vinay944924
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-01-29 at 3 35 38 PM

Related PRS (if any):

This frontend PR is related to the development backend PR.
The Previous Frontend PR is #3733

Main changes explained:

  • Updated the Project risk Profile Overview Jsx and Css file for the graph.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as any user
  5. go to http://localhost:5173/bmdashboard/totalconstructionsummary
  6. Open the "Risk profiles for projects" section and verify that you can view the bar graph as shown below

Screenshots or videos of changes:

Screenshot 2026-01-29 at 3 40 57 PM

Note:

The Previous backend PR was not working so you can test this PR by using development branch for the Backend.

@netlify
Copy link
Copy Markdown

netlify bot commented Jan 29, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit fcec86b
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69e682441dce8d0008a415c5
😎 Deploy Preview https://deploy-preview-4771--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
Copy Markdown

@Vinay944924 Vinay944924 changed the title Vinay_K_Project_Risk_Profile Vinay K Project Risk Profile Mar 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 Apr 9, 2026
@Nirali164
Copy link
Copy Markdown

Hello Vinay,
I have checked this branch locally. In dark mode, the labels are not visible. the dropdown placeholder
Screenshot 2026-04-09 at 2 46 09 PM
It looks fine in light mode
Screenshot 2026-04-09 at 2 47 04 PM

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 Vinay,

I tested this PR locally. The chart renders correctly and the page is responsive. However, the Project and Dates dropdown menus are not properly aligned in dark mode, the dropdown items appear with a light background instead of matching the dark theme.

Image Image Image

Copy link
Copy Markdown

@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 Vinay,

I have reviewed your PR locally and below are the issues i found as shown highlighted in the video.

  • The dropdown, do not go back unless we click on them when all options are unselected.
  • In dark mode, the dropdown is in light mode.
  • The labels project title is very light compared to the background.
4771.mov

@Anusha-Gali Anusha-Gali self-requested a review April 14, 2026 19:49
Copy link
Copy Markdown

@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.

Have submitted approved by mistake, apologies - requesting changes for the above mentioned issues.

@sonarqubecloud
Copy link
Copy Markdown

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.

5 participants