Skip to content

Update bar graph tooltip#810

Merged
shinyichen merged 1 commit intoadsabs:masterfrom
shinyichen:SCIX-832-bar-graph-tooltip-update
Feb 24, 2026
Merged

Update bar graph tooltip#810
shinyichen merged 1 commit intoadsabs:masterfrom
shinyichen:SCIX-832-bar-graph-tooltip-update

Conversation

@shinyichen
Copy link
Member

Update the Bar graph tooltip to show all groups

Screenshot 2026-02-24 at 10 22 47 AM

Copy link
Member

@thostetler thostetler left a comment

Choose a reason for hiding this comment

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

LTGM!

@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 19.23077% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.0%. Comparing base (fe8e34d) to head (5f22482).

Files with missing lines Patch % Lines
src/components/Visualizations/Graphs/BarGraph.tsx 20.6% 31 Missing ⚠️
...mponents/Visualizations/GraphPanes/MetricsPane.tsx 9.1% 10 Missing ⚠️
...nents/Visualizations/GraphPanes/YearsGraphPane.tsx 50.0% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #810     +/-   ##
========================================
- Coverage    62.0%   62.0%   -0.0%     
========================================
  Files         317     317             
  Lines       36490   36536     +46     
  Branches     1642    1642             
========================================
+ Hits        22614   22618      +4     
- Misses      13839   13881     +42     
  Partials       37      37             
Files with missing lines Coverage Δ
...nents/Visualizations/GraphPanes/YearsGraphPane.tsx 11.5% <50.0%> (-<0.1%) ⬇️
...mponents/Visualizations/GraphPanes/MetricsPane.tsx 15.6% <9.1%> (-<0.1%) ⬇️
src/components/Visualizations/Graphs/BarGraph.tsx 18.7% <20.6%> (-0.9%) ⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shinyichen shinyichen merged commit 325ee16 into adsabs:master Feb 24, 2026
5 checks passed
@shinyichen shinyichen deleted the SCIX-832-bar-graph-tooltip-update branch February 24, 2026 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants