IntenseWebs/grafana
3
0
Fork 0
mirror of https://github.com/grafana/grafana.git synced 2025-02-25 18:55:37 -06:00
Code Issues Packages Projects Releases Wiki Activity

datatrails: ensure breakdown yaxis sync is maintained when layout changes (#85380)

Browse Source 
fix: ensure breakdown yaxis sync is maintained when layout changes
This commit is contained in:
Darren Janeczek
2024-04-10 17:12:33 -04:00
committed by GitHub
parent 486c956098
commit 2e1e45bbb1
1 changed files with 17 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
public/app/features/trails/ActionTabs/BreakdownScene.tsx
View File

@@ -80,16 +80,28 @@ export class BreakdownScene extends SceneObjectBase<BreakdownSceneState> {
}
});
const metricScene = sceneGraph.getAncestor(this, MetricScene);
const metric = metricScene.state.metric;
this._query = getAutoQueriesForMetric(metric).breakdown;
// The following state changes (and conditions) will each result in a call to `clearBreakdownPanelAxisValues`.
// By clearing the axis, subsequent calls to `reportBreakdownPanelData` will adjust to an updated axis range.
// These state changes coincide with the panels having their data updated, making a call to `reportBreakdownPanelData`.
// If the axis was not cleared by `clearBreakdownPanelAxisValues` any calls to `reportBreakdownPanelData` which result
// in the same axis will result in no updates to the panels.
const trail = getTrailFor(this);
trail.state.$timeRange?.subscribeToState(() => {
// The change in time range will cause a refresh of panel values,
// so we clear the axis range so it can be recalculated when the calls
// to `reportBreakdownPanelData` start being made from the panels' behavior.
// The change in time range will cause a refresh of panel values.
this.clearBreakdownPanelAxisValues();
});
const metric = sceneGraph.getAncestor(this, MetricScene).state.metric;
this._query = getAutoQueriesForMetric(metric).breakdown;
metricScene.subscribeToState(({ layout }, old) => {
if (layout !== old.layout) {
// Change in layout will set up a different set of panel objects that haven't received the current yaxis range
this.clearBreakdownPanelAxisValues();
}
});
this.updateBody(variable);
}