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

Chore: Remove viz panel tooltip related code that is no longer needed #675

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

kaydelaney
Copy link
Contributor

@kaydelaney kaydelaney commented Apr 3, 2024

This used to be needed for the new viz tooltips, but should no longer be needed now that they're portaled.

This also fixes an issue where panels lower down a page always appeared beneath ones above when being moved around.

See also: grafana/grafana#85563

Closes #622

📦 Published PR as canary version: 4.3.1--canary.675.8546552145.0

✨ Test out this PR locally via:

npm install @grafana/scenes@4.3.1--canary.675.8546552145.0
# or 
yarn add @grafana/scenes@4.3.1--canary.675.8546552145.0

@kaydelaney kaydelaney added type/bug Something isn't working patch Increment the patch version when merged release Create a release when this pr is merged labels Apr 3, 2024
@kaydelaney kaydelaney self-assigned this Apr 3, 2024
@kaydelaney kaydelaney merged commit a75b11d into main Apr 4, 2024
3 checks passed
@kaydelaney kaydelaney deleted the remove-descending-z branch April 4, 2024 09:25
@grafanabot
Copy link
Contributor

🚀 PR was released in v4.4.1 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged release Create a release when this pr is merged released type/bug Something isn't working
Projects
Status: 🚀 Done
Development

Successfully merging this pull request may close these issues.

revert LayoutItemContext and DescendingZindex
3 participants