Aseem - Phase 2 - Add filters for for Predicted vs Actual Cost Line chart and UI changes#5375
Aseem - Phase 2 - Add filters for for Predicted vs Actual Cost Line chart and UI changes#5375aseemdeshmukh wants to merge 6 commits into
Conversation
…tom tool tip Co-authored-by: Copilot <copilot@github.com>
✅ 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 Aseem,
I tested this PR locally. The Financial Tracking features, including tooltips, project and date filters, reset functionality, custom tooltips, pie chart values and percentages, cost variance indicators, and consistent color mapping, are working as expected in dark mode.
However, the calendar is not properly aligned in dark mode, and the SonarQube check is failing. Please review these issues before merging.
beblicarl
left a comment
There was a problem hiding this comment.
This feature works as intended and the code is great. However, you have some merge conflicts that needs to be resolved
|
Closing this PR and created a new PR #5403 |


Description
This PR covers/solves both of these similar tasks:

And also this task:
Related PRS (if any):
This is a redo of PR #4354 Frontend PR, including fixes for all the feedback provided there.
Main changes explained:
How to test:
a) For dark mode, tooltips and texts are readable and can be distinguished --> * tooltip explanations for Planned, Actual, and Predicted costs exist
b) Filters for Projects and date filter work
c) reset filter button works
d) custom tooltip is working
Screenshots or videos of changes:
1.mp4