Skip to content

Feat tooltip stacked area chart - #909

Merged
jacobvjk merged 3 commits into
mainfrom
feat-tooltip-stacked-area-chart
Aug 19, 2026
Merged

Feat tooltip stacked area chart#909
jacobvjk merged 3 commits into
mainfrom
feat-tooltip-stacked-area-chart

Conversation

@jacobvjk

Copy link
Copy Markdown
Collaborator

Summary

To make the overall experience of benchmark charts more consistent across plot types, this PR:

  • adds a hover tooltip to NormalizedStackedAreaChart showing all technology shares for the nearest year, with a vertical guideline marking it
  • ensures the tooltip sits beside the added vertical guideline and auto-flips/repositions to stay within the chart bounds
  • slightly enlarges the comparison view's 3-pathway chart size to give the tooltip more room

Related issues

Closes: #908

This is also a preparation for #904

Testing

Manually tested the plot behavior, none of the plot components currently have dedicated tests

Checklist

  • PR is focused on a single concern
  • Tests pass locally and in CI
  • Docs updated for user-visible changes
  • AI-assisted portions declared (see CONTRIBUTING.md)

Copilot AI lite review requested due to automatic review settings August 18, 2026 16:19
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown

Expected version change and release notes:

1.16.0-dev.15 (v1.16.0-dev.14...feat-tooltip-stacked-area-chart ) (2026-08-19T07:22 UTC)

Features

  • add tooltip to NormalizedStackedAreaChart (ee13009)

@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-glacier-0f640931e-909.westus2.2.azurestaticapps.net

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves interaction consistency across benchmark plot types by adding an x-axis-driven hover tooltip to NormalizedStackedAreaChart, including a vertical guideline and tooltip positioning logic to stay within bounds. It also increases the 3-pathway comparison chart dimensions to give the new tooltip more room.

Changes:

  • Add pointer-driven tooltip + vertical guideline to NormalizedStackedAreaChart, showing nearest-year shares for all technologies.
  • Implement tooltip auto-flip/reposition logic to keep the tooltip within the chart area while keeping the guideline visible.
  • Increase the 3-pathway comparison panel dimensions to reduce tooltip crowding.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/components/NormalizedStackedAreaChart.tsx Adds guideline + hover tooltip that lists all technology shares for the nearest year and repositions to avoid clipping.
src/components/ComparisonPlots.tsx Enlarges 3-pathway comparison chart dimensions to provide more space for the new tooltip.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jacobvjk
jacobvjk requested a review from AlexAxthelm August 18, 2026 16:25
Copilot AI review requested due to automatic review settings August 19, 2026 07:21
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-glacier-0f640931e-909.westus2.2.azurestaticapps.net

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@jacobvjk
jacobvjk merged commit b9c5dd2 into main Aug 19, 2026
13 checks passed
@jacobvjk
jacobvjk deleted the feat-tooltip-stacked-area-chart branch August 19, 2026 09:24
@jacobvjk jacobvjk mentioned this pull request Aug 19, 2026
4 tasks
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.

Plots: Add tooltip to stacked area chart

3 participants