Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove old tooltips and annotations #83632

Closed
4 of 51 tasks
leeoniya opened this issue Feb 28, 2024 · 1 comment · Fixed by #84420
Closed
4 of 51 tasks

Remove old tooltips and annotations #83632

leeoniya opened this issue Feb 28, 2024 · 1 comment · Fixed by #84420
Assignees
Labels
area/tooltip internal for issues made by grafanistas type/epic Issue made of smaller issues

Comments

@leeoniya
Copy link
Contributor

leeoniya commented Feb 28, 2024

before we can remove old tooltips, we need to deploy newVizTooltips in HG, and validate parity with old tooltips.

this is a sub-epic of #68238

we have started rollout of this feature toggle internally to dev and staging for dogfooding and feedback: https://github.com/grafana/deployment_tools/pull/124451

DRY

Verify datalinks work correctly in all panels

Verify exemplars tooltip works correctly

  • TimeSeries
  • Heatmap

Verify other plugins play nice

Remove old tooltip code from panels

  • StateTimeline
  • StatusHistory
  • BarChart
  • Trend
  • Candlestick
  • TimeSeries
  • Heatmap
  • XYChart

Remove now-unused components and features

unserious-but-good reference: #83557

@leeoniya leeoniya added type/epic Issue made of smaller issues area/tooltip internal for issues made by grafanistas labels Feb 28, 2024
@leeoniya leeoniya added this to the 11.0.x milestone Feb 28, 2024
@nmarrs
Copy link
Contributor

nmarrs commented Feb 28, 2024

holy issue description batman 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tooltip internal for issues made by grafanistas type/epic Issue made of smaller issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants