[UI/UX] Chart Clarity Review: PRMergeSuccessRateChart
Context
The PRMergeSuccessRateChart compares lifetime versus sprint data using a percentage-based Y-axis. The chart includes a custom hover implementation for its "info" icon. We need a developer to review the chart's visual hierarchy, verify the percentage calculations render cleanly, and ensure the custom hover state behaves nicely across different screen sizes.
Acceptance Criteria
Files to Check:
src/components/charts/PRMergeSuccessRateChart.jsx
[UI/UX] Chart Clarity Review: PRMergeSuccessRateChart
Context
The
PRMergeSuccessRateChartcompares lifetime versus sprint data using a percentage-based Y-axis. The chart includes a custom hover implementation for its "info" icon. We need a developer to review the chart's visual hierarchy, verify the percentage calculations render cleanly, and ensure the custom hover state behaves nicely across different screen sizes.Acceptance Criteria
PRMergeSuccessRateChart.jsx.onMouseEnter/onMouseLeaveinfo tooltip implementation. Does it clip off the edge of the screen on smaller monitors? Consider if this needs to be refactored into a standard CSS hover class or a reusable Tooltip component to maintain project coding standards.33.333333%).#4F46E5) and "Sprint" (#22C55E) bars.Files to Check:
src/components/charts/PRMergeSuccessRateChart.jsx