Juhitha fix people report page UI#4635
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Juhitha,
While testing in dark mode, I noticed a couple of small UI things:
- The page header text and person names were a bit hard to read due to low contrast.
- The profile toggle/icon area on the right side also looks slightly off in dark mode. The circle outline and alignment feel a little inconsistent with the rest of the theme.
Could you please take a look at these in dark mode and adjust the styling for better consistency and contrast?
Thanks! Let me know once updated.
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Juhitha,
Thank you for working on the People Report page UI updates. I tested the PR locally by following the provided steps. Most of the layout changes look good.
However, in dark mode, the toggle under the hours donut chart has text appearing behind it, which affects visibility. Also, the Time Entries graph section appears to have a light background that does not match the rest of the dark theme. Please confirm if the graph styling is part of this PR.
|
Hi @HemanthNidamanuru, Happy to discuss further if needed! |
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Here's the updated comment:
Hi Juhitha, thank you for working on these improvements! I reviewed your PR locally and while most of the UI requirements are met, I noticed a couple of issues:
- In dark mode, the headings in "Tasks with Completed Hours" are not clearly visible
- The dropdowns in "Tasks Contributed" appear misaligned
- The profile picture is not displaying in a circle shape
- The background is still appearing in light mode when dark mode is enabled
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Juhitha,
I suppose the fixes broke the working functionality. Below are the issues found:
- The profile picture is not displaying in a circle shape and the active/inactive circle does not seem to exist beside the picture
- The dropdowns in "Tasks Contributed" appear misaligned and do not seem to work
- Few of the dropdowns in "Tasks Contributed" are not visible in dark mode
- The background is still appearing in light mode when dark mode is enabled
- In dark mode, the headings in "Tasks with Completed Hours" are not clearly visible
- The toggle is not visible in "Time Logged to Projects chart" in dark mode
- The toggle is not visible in "Tasks with Completed Hours"
- Mobile resolution needs fixing mainly in both the chart tabular data
|
Hi @Anusha-Gali, I have reverted the changes, as the previous description was a bit misleading. I believe these items are now resolved, along with the previously mentioned points. |
|
@Anusha-Gali , @HemanthNidamanuru The PR description was a bit misleading, and I had added a few points that were not part of the original requirements. Please refer to this comment for the correct ticket description. |
|
Submitting request for changes as the previous approval was made by mistake.
rithika-paii
left a comment
There was a problem hiding this comment.
Hi Juhita,
I reviewed PR #4635 and tested the People Report page locally in light mode, dark mode, desktop, and mobile views. The overall responsiveness and styling improvements work well, but I found a couple of issues/suggestions:
1. Filter/Search input issue: On my side, the text box is not accepting typed input properly, so I’m unable to use the field as expected. (See attached video)
PR.4635.-.Light.mode.changes.mov
2. Mobile alignment: The mobile view is generally responsive and usable, but some alignment could still be improved for a cleaner layout. (See attached video.)
PR.4635.-.Mobile.view.-.dark.mode.changes.mov
3. Bottom report option alignment: The 7 report option buttons/sections at the bottom (People / Teams / Projects area) do not appear well-aligned visually. Functionally, they work, but the layout feels uneven and could be improved for better UI consistency. (See attached screenshot.)
kunchalasireesha
left a comment
There was a problem hiding this comment.
Hi Juhitha, it looks like this PR currently has merge conflicts with the base branch. Could you please rebase or merge the latest changes from development into your branch and resolve the conflicts? Once the build passes, I’ll finish the review.























Description
Fixed People Report page UI issues for mobile and narrow screens (375px and up). Improved table layout, column widths, and added comprehensive dark mode support for mobile card view and filter controls.
Fixes WBS: Fix PEOPLE REPORT PAGE UI issues for 375px and up (Juhitha Reddy Penumalli)
Related PRS (if any):
Reference to Previous PR #2005, PR #2650
Main changes explained:
PeopleTableDetails.css→PeopleTableDetails.module.cssto comply with project CSS modules conventionPeopleTableDetails.jsxto use CSS modules syntax for all className references2.2frto3.5frto display longer task names properly1frto0.8fr) for better space utilizationjustify-content: space-betweengap: 0to grid layout8px 16px#232946)#1C2541)#5A67D8)How to test:
Check out current branch:
Juhitha_fix_people_report_page_UIRun
npm installto ensure all dependencies are installedClear site data/cache in browser (DevTools → Application → Clear storage)
Run the app locally:
npm startLog in as Admin or Owner user
Navigate to: Reports → Reports → People tab
Click on any user name to view their report
Verify Desktop View (> 1024px):
Verify Mobile View (< 1024px):
Verify Dark Mode:
#232946)#1C2541)Verify Filters:
Screenshots or videos of changes:
Uploading Screen Recording 2025-12-19 at 1.48.58 PM.mov…
Desktop View - Light Mode:
Mobile View - Dark Mode:
Clear Filters Button: