Adithya Improve Expense Chart#5301
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,
I tested this PR locally and found the following issues:
Variance color coding not visible in dark mode - the color coded variance text on hover is not visible/readable in dark mode.
Calendar misaligned in dark mode - the date picker calendar is not properly aligned when dark mode is enabled.
Future dates selectable in calendar - users can select future dates in the date filter, which should not be allowed for expense data.
Hi @HemanthNidamanuru, thank you for the feedback! I have pushed a new commit addressing all of your points: Variance Colors: Updated the dark mode theme variables to use high-contrast neon red/green so the text is fully readable on hover and above the bars. Calendar Alignment: Refactored the filter container to use a clean 2x2 grid, ensuring the End Date input no longer wraps awkwardly. Added color-scheme support so the native calendar renders properly in dark mode. Future Dates: Fixed the date logic to use the user's local timezone (preventing UTC offset bugs) and applied it to the max attribute, successfully disabling all future dates. Can you please re-review this pr and approve it. Thank you. |
There was a problem hiding this comment.
- Tested the PR locaaly and Planned and Actual cost works as expected.
- The Previous issues are issues are resolved and tooltip, calendar and filters are working as expected.
- But Cost Breakdown by Category as show in the last screenshot doesn't display the From and To dates properly in dark mode. Is this scoped for this PR?
Hi @DeepighaJ, Thank you so much for testing this and confirming that the tooltip, calendar, and filter fixes are working perfectly! Regarding the "Cost Breakdown by Category" dates in dark mode: You are exactly right, that is outside the scope of this PR. This ticket was strictly focused on rebuilding the "Planned vs Actual Cost" graph. To avoid scope creep and prevent potential merge conflicts with any other developers working on the Financials dashboard, I purposefully haven't modified the neighboring CostBreakDown component. I highly recommend we open a separate ticket to track and fix the dark mode date display for that specific chart. Since the in-scope items are functioning as expected, could you please approve this PR so we can get these improvements merged? Thank you again for your help! |
Thank you for testing the latest changes! I have just pushed an update to address the dark mode tooltip issue. Could you please pull the latest changes, clear your cache, and test the hover colors in dark mode once more? Let me know if everything looks good on your end! |
|
|
Thank you all, merging! |

















Description
This PR enhances the "Planned vs Actual Cost" graph on the Financials dashboard to provide immediate, actionable insights into project budgets. It introduces dynamic variance calculations, custom tooltips, and clear visual indicators for over/under-budget status.
Related PRs (if any):
Main changes explained:
darkModestate from the parent component.prop-typesvalidation foractive,payload,label,darkMode, and nested payload structures to ensure data reliability.How to test:
git checkout adithya_improve_expense_chartnpm installandnpm run start:local.Screenshots or videos of changes:
Fix.Actual.vs.planned.cost.graph.visuals.mp4