Alisha - fix(job-analytics): improve tooltip data clarity for application time…#5043
Alisha - fix(job-analytics): improve tooltip data clarity for application time…#5043alishawalunj wants to merge 3 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.
There was a problem hiding this comment.
Looks good overall,
Just one accessibility suggestion: the .bar div is interactive (hover/tooltip), so it should ideally support keyboard access (e.g., tabIndex, role="button", and focus/blur handlers).
Also noticed a few duplicate CSS properties that could be cleaned up.
Otherwise, great work on the tooltip behavior and positioning
4cb7b27
062381f to
4cb7b27
Compare
|
rithika-paii
left a comment
There was a problem hiding this comment.
Hi Alisha,
I reviewed the PR #5043 locally, and it works well! No changes required from my side!















Description
Related PRS (if any):
To test this frontend PR you need to checkout development branch for backend.
Main changes explained:
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes: