Dynamic Dashboards: Make outline open by default (#114146)
This commit is contained in:
@@ -22,8 +22,6 @@ test.describe(
|
||||
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.NavToolbar.editDashboard.editButton).click();
|
||||
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.section).click();
|
||||
|
||||
// Should be able to click Variables item in outline to see add variable button
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.item('Variables')).click();
|
||||
await expect(
|
||||
|
||||
@@ -199,7 +199,6 @@ test.describe(
|
||||
.click();
|
||||
|
||||
// Open the modal editor in the side pane
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.section).click();
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.node('Variables')).click();
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.item('foo')).click();
|
||||
await openModal(dashboardPage, selectors);
|
||||
|
||||
@@ -48,7 +48,6 @@ export const flows = {
|
||||
},
|
||||
async newEditPaneVariableClick(dashboardPage: DashboardPage, selectors: E2ESelectorGroups) {
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.NavToolbar.editDashboard.editButton).click();
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.section).click();
|
||||
await dashboardPage.getByGrafanaSelector(selectors.components.PanelEditor.Outline.item('Variables')).click();
|
||||
await dashboardPage
|
||||
.getByGrafanaSelector(selectors.components.PanelEditor.ElementEditPane.addVariableButton)
|
||||
|
||||
@@ -35,7 +35,7 @@ export function DashboardEditPaneRenderer({ editPane, isEditPaneCollapsed, onTog
|
||||
const isNewElement = selection?.isNewElement() ?? false;
|
||||
const [outlineCollapsed, setOutlineCollapsed] = useLocalStorage(
|
||||
'grafana.dashboard.edit-pane.outline.collapsed',
|
||||
true
|
||||
false
|
||||
);
|
||||
const [outlinePaneSize = 0.4, setOutlinePaneSize] = useLocalStorage('grafana.dashboard.edit-pane.outline.size', 0.4);
|
||||
|
||||
|
||||
@@ -162,7 +162,7 @@ describe('NavToolbarActions', () => {
|
||||
await userEvent.click(await screen.findByTestId(selectors.components.NavToolbar.editDashboard.editButton));
|
||||
expect(DashboardInteractions.editButtonClicked).toHaveBeenCalledWith({
|
||||
dashboardUid: 'dash-1',
|
||||
outlineExpanded: false,
|
||||
outlineExpanded: true,
|
||||
});
|
||||
});
|
||||
|
||||
|
||||
+1
-1
@@ -58,7 +58,7 @@ describe('EditDashboardSwitch', () => {
|
||||
it('should call DashboardInteractions.editButtonClicked with outlineExpanded:true if grafana.dashboard.edit-pane.outline.collapsed is undefined', async () => {
|
||||
render(<EditDashboardSwitch dashboard={buildTestScene()} />);
|
||||
await userEvent.click(await screen.findByTestId(selectors.components.NavToolbar.editDashboard.editButton));
|
||||
expect(DashboardInteractions.editButtonClicked).toHaveBeenCalledWith({ outlineExpanded: false });
|
||||
expect(DashboardInteractions.editButtonClicked).toHaveBeenCalledWith({ outlineExpanded: true });
|
||||
});
|
||||
|
||||
it('should call DashboardInteractions.editButtonClicked with outlineExpanded:true if grafana.dashboard.edit-pane.outline.collapsed is false', async () => {
|
||||
|
||||
+1
-1
@@ -59,7 +59,7 @@ describe('MakeDashboardEditableButton', () => {
|
||||
it('should call DashboardInteractions.editButtonClicked with outlineExpanded:true if grafana.dashboard.edit-pane.outline.collapsed is undefined', async () => {
|
||||
render(<MakeDashboardEditableButton dashboard={buildTestScene()} />);
|
||||
await userEvent.click(await screen.findByTestId(selectors.components.NavToolbar.editDashboard.editButton));
|
||||
expect(DashboardInteractions.editButtonClicked).toHaveBeenCalledWith({ outlineExpanded: false });
|
||||
expect(DashboardInteractions.editButtonClicked).toHaveBeenCalledWith({ outlineExpanded: true });
|
||||
});
|
||||
|
||||
it('should call DashboardInteractions.editButtonClicked with outlineExpanded:true if grafana.dashboard.edit-pane.outline.collapsed is false', async () => {
|
||||
|
||||
@@ -46,7 +46,7 @@ export const trackDeleteDashboardElement = (element: EditableDashboardElementInf
|
||||
|
||||
export const trackDashboardSceneEditButtonClicked = (dashboardUid?: string) => {
|
||||
DashboardInteractions.editButtonClicked({
|
||||
outlineExpanded: !store.getBool('grafana.dashboard.edit-pane.outline.collapsed', true),
|
||||
outlineExpanded: !store.getBool('grafana.dashboard.edit-pane.outline.collapsed', false),
|
||||
dashboardUid,
|
||||
});
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user