Skip to content

Neeraj Fix Hiring Analytics chart rendering and improve layout styling#4896

Open
Neeraj-Kondaveeti wants to merge 1 commit intodevelopmentfrom
Neeraj_Fix_HiringAnalytics_Chart
Open

Neeraj Fix Hiring Analytics chart rendering and improve layout styling#4896
Neeraj-Kondaveeti wants to merge 1 commit intodevelopmentfrom
Neeraj_Fix_HiringAnalytics_Chart

Conversation

@Neeraj-Kondaveeti
Copy link
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented Feb 25, 2026

Description

image

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

This PR improves the clarity, accuracy, and usability of the “Number of People Hired vs. Total Applications” chart.

  1. Chart Accuracy & Label Fixes
  • Corrected misleading axis labeling.
  • X-axis now dynamically updates:
    • "Number of Applications / Hires" in Count View
    • "Percentage of People Hired (%)" in Percentage View
  • Y-axis label updated to "Role".
  1. Added Percentage Toggle View
  • Implemented toggle between:
    • Count View
    • Percentage View
  • Percentage is calculated as:
    (totalHired / totalApplicants) * 100
  • Values rounded to 1 decimal place.
  1. UX & Readability Improvements
  • Added clear legend for metric differentiation.
  • Adjusted spacing (barSize, barCategoryGap, margins) to prevent label overlap.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. Navigate to: http://localhost:5173/applicant-volunteer-ratio
  6. Verify:
    • Count View:

      • Blue = Total Applications
      • Green = People Hired
      • X-axis shows "Number of Applications / Hires"
      • Values display correctly without overlap
    • Percentage View:

      • Chart switches to percentage mode
      • X-axis shows "Percentage of People Hired (%)"
      • Values include % symbol
      • Domain is 0–100
    • Role Filtering:

      • Selecting/removing roles updates chart correctly

Screenshots or videos of changes:

image

Note:

  • Edge cases validated (invalid date range, empty role selection, zero data).

@netlify
Copy link

netlify bot commented Feb 25, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 5bf8f19
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/699f7c7aee002400088253cb
😎 Deploy Preview https://deploy-preview-4896--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link

@Neeraj-Kondaveeti Neeraj-Kondaveeti changed the title Fix Hiring Analytics chart rendering and improve layout styling Neeraj Fix Hiring Analytics chart rendering and improve layout styling Feb 25, 2026
Copy link

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Neeraj,

Well done with this implementation.

Image Image Image

Copy link

@Vishnu10-hub Vishnu10-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image Image Image
  • Checked locally with backend running.

  • Both Count and Percentage views display correctly.

  • Values and axes render properly.

  • Role filtering updates the chart as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants