oss-slu / oss-slu/oss_dev_analytics
Chart Clarity Review: PRMergeSuccessRateChart
Open
@kiaramathews is already working on this.
Since May 5, 2026.
frontend
- Dominant language
- JavaScript
- Stars
- 2
- Forks
- 2
- Avg merge
- 1m
- Merged PRs (30d)
- 1
Description
[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
- Review
PRMergeSuccessRateChart.jsx. - Test the custom
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. - Verify that the Y-axis and chart tooltips format the percentages correctly (e.g., ensuring no long decimal trails like
33.333333%). - Check the color contrast between the "Lifetime" (
#4F46E5) and "Sprint" (#22C55E) bars. - Submit a PR to refactor any confusing UI elements, tooltip behavior, or styling.
Files to Check:
src/components/charts/PRMergeSuccessRateChart.jsx
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.