Open
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
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.



Improve "Select Projects" Form UX in Building Summary Section (BM Dashboard)
Description
Fixes usability issues within the BM Dashboard's Building Summary section where the "Select Projects" dropdown provided minimal feedback and lacked clear interaction cues. Users can now easily understand their selection state and see clear visual feedback when the summary updates.
Related PRS (if any):
This frontend PR utilizes existing backend endpoints. No new backend PR is required.
Main changes explained:
ProjectsList.jsx):Selectcomponent in aFormGroupwith aLabeland helper<small>text. Added the requested placeholder.isUpdatingstate andsetTimeout(600ms) to manage the Spinner transition, list opacity, and disable the dropdown during changes.<h3>header to reflect current selections. Fixed list bleeding by applyinglistStyleType: 'none'and adjusted margins for cleaner card spacing.ProjectStatusDonutChart.jsx&ProjectStatusDonutChart.module.css):darkModestate. Applied a new.darkContainerclass to fix stark white backgrounds and update date picker colors (color-scheme: dark).contentStyleanditemStyleto the Recharts<Tooltip />so text is readable on hover in both light and dark modes.aspect={1}withheight={350}on theResponsiveContainerto prevent the chart from infinitely scaling on large screens.BMDashboard.jsx):<ProjectStatusDonutChart />to resolve SonarCloud warnings regarding unused/empty data states as per the diff.How to test:
feature/bm-dashboard-select-projects-ux).npm installandnpm run start:local(ornpm run dev) to run this PR locally.Screenshots or videos of changes:
Screen.Recording.mp4