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
Dashboard: Revert LayoutItemContext #83465
Conversation
grafana-uiRemovalsLayoutItemContext/home/runner/work/grafana/grafana/base/grafana-ui/dist/index.d.ts LayoutItemContext LayoutItemContextProps /home/runner/work/grafana/grafana/base/grafana-ui/dist/index.d.ts LayoutItemContextProps LayoutItemContextProps.boostZIndex /home/runner/work/grafana/grafana/base/grafana-ui/dist/index.d.ts boostZIndex(): () => void; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there were/are some breaking changes check issues in drone? But other than that, this works as expected in all tested panel types.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I like that we are removing things and making the API surface smaller 🙌🏻
I noticed that we use this in scenes. What it the replacement for LayoutItemContext? Do we have any or are they left on their own to figure this out? I guess that applies to potential external plugins as well.
@leeoniya this requires some kind of revert of grafana/scenes#530 as well, do you mind picking this up? |
that pr has more changes in it that im not familiar with. was hoping you peeps could pick it up: grafana/scenes#622 also waiting in scenes update to land |
we've switched to using global portals in #81986, so this is no longer needed.
originally introduced in #80116