Adithya Enhanced Popularity Timeline Chart Frontend#4416
Adithya Enhanced Popularity Timeline Chart Frontend#4416one-community merged 12 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Hi Aditya,
I have reviewed your PR locally and the functionalities do work as per requirement but there are two UI issues which i have mentioned below.

The word "Show Low Volume" is not visible in dark mode.

The Filter by Role dropdown options are not visible in dark mode and does not adapt to dark mode

Hi @Anusha-Gali, thanks for the review. I've updated the CSS for the "Show Low Volume" label and added styling to the Role dropdown to fix the visibility issues in dark mode. Ready for a re-review when you have a chance. |
bhanuanishakkineni
left a comment
There was a problem hiding this comment.
Hello Adithya,
I have reviewed and tested both frontend and backend of this PR. Please find the backend comments at #1885.
I directed to http://localhost:5173/popularity-enhanced page and tested the functionality of role-based popularity timeline chart.
The filters are working and I am able to choose single or multiple roles, time range.
The reset filters button and show low volume checkbox are also working fine. UI is consistent with both light and dark modes.
I am also able to highlight a specific role in timeline using the cards below for individual role.
Below are the UI issues I found,
- Since there are to lines (solid and dotted) for each role, its better to distinguish between hits and applications lines in legend for better UX.
- While hovering over the months axis(find it from testing video below) in the timeline chart, if user wants to check all roles' data from a month, its hard for the user to scroll down on the tooltip while there are more number of months as the tooltip is changing immediately because of near by month.
Try to implement feature that if mouse hovered on tooltip, event listeners from the chart shouldn't work. - For small screen sizes(Mobile) the graph is being compressed and so does the months on x-axis. Try to make the graph scrollable for small screen so that it will be readable.
PR.4416.1885.testing.mp4
Hi @bhanuanishakkineni, thanks for the review! I have pushed the fixes for the UI issues you noted:
Ready for re-review! |
bhanuanishakkineni
left a comment
There was a problem hiding this comment.
Hello Adithya,
I have reviewed changes made to this PR.
Legend added.
Tooltip working fine.
Graph is readable for mobile screens.
I am approving this PR
|
|
Thank you all, merging! |
















Description
Implements the Enhanced Role-Based Popularity Analytics dashboard for the Job Posting Analytics page. It replaces the previous basic line chart with a comprehensive visualization that compares "Hits" versus "Applications" for multiple job roles simultaneously.
The chart supports filtering by dynamic date ranges (3, 6, 12 months) and job roles, displays dual lines (solid for Hits, dashed for Applications), includes interactive highlighting via the legend/summary cards, and adapts to dark mode.
Related PRS (if any)
This frontend PR is related to backend PR Adithya - Enhanced Popularity Analytics Backend. To test this frontend PR, checkout backend PR Adithya - Enhanced Popularity Analytics Backend.
Main changes explained:
EnhancedPopularityTimelineChart.jsxto render the advanced line chart with Recharts.How to test:
npm install/npm starton frontend.envfile is configured as:Note: Use http://localhost:4500/api instead of the staging endpoint (https://api-staging.highestgood.com/api) to see the graph working.
Screenshots or videos of changes:
Enhanced.Popularity.Timeline.mp4
Note:
Please mute the audio of this recording. Timer sounds have been recorded.