Skip to content

Commit 7587681

Browse files
nreesekibanamachineelasticmachine
authored
[dashboard] remove legacy section from 'Add panel' flyout (#208116)
Users should use Lens to visualize data. Users should not use TSVB or aggregation based panels. To this end, "easy button" UIs, such as dashboard's "Add panel" flyout, should guide users to lens and not guide them to legacy applications. This PR removes `legacy` section from dashboard "Add panel" flyout. Users can still add legacy panels to dashboards via "visualize" application. The path is more steps, but this is as intended. Using old stuff should be painful and require extra work as a caret to move users to new stuff. <img width="500" alt="Screenshot 2025-01-23 at 1 08 41 PM" src="https://github.com/user-attachments/assets/e142cba9-a880-4ad6-a3f8-a981a03daaec" /> --------- Co-authored-by: kibanamachine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
1 parent 2212a19 commit 7587681

File tree

20 files changed

+108
-407
lines changed

20 files changed

+108
-407
lines changed

src/platform/plugins/shared/dashboard/public/dashboard_app/top_nav/add_panel_button/get_menu_item_groups.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import { VisGroups } from '@kbn/visualizations-plugin/public';
1111
import { ADD_PANEL_TRIGGER } from '@kbn/ui-actions-plugin/public';
1212
import {
1313
ADD_PANEL_ANNOTATION_GROUP,
14-
ADD_PANEL_LEGACY_GROUP,
1514
ADD_PANEL_OTHER_GROUP,
1615
ADD_PANEL_VISUALIZATION_GROUP,
1716
} from '@kbn/embeddable-plugin/public';
@@ -23,11 +22,9 @@ import { uiActionsService, visualizationsService } from '../../../services/kiban
2322
import { navigateToVisEditor } from './navigate_to_vis_editor';
2423
import type { MenuItem, MenuItemGroup } from './types';
2524

26-
const VIS_GROUP_TO_ADD_PANEL_GROUP: Record<VisGroups, undefined | PresentableGroup> = {
27-
[VisGroups.AGGBASED]: undefined,
25+
const VIS_GROUP_TO_ADD_PANEL_GROUP: Record<string, PresentableGroup> = {
2826
[VisGroups.PROMOTED]: ADD_PANEL_VISUALIZATION_GROUP,
2927
[VisGroups.TOOLS]: ADD_PANEL_ANNOTATION_GROUP,
30-
[VisGroups.LEGACY]: ADD_PANEL_LEGACY_GROUP,
3128
};
3229

3330
export async function getMenuItemGroups(

src/platform/plugins/shared/embeddable/public/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,5 @@ export function plugin(initializerContext: PluginInitializerContext) {
5656
export {
5757
ADD_PANEL_ANNOTATION_GROUP,
5858
ADD_PANEL_OTHER_GROUP,
59-
ADD_PANEL_LEGACY_GROUP,
6059
ADD_PANEL_VISUALIZATION_GROUP,
6160
} from './ui_actions/add_panel_groups';

src/platform/plugins/shared/embeddable/public/ui_actions/add_panel_groups.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,3 @@ export const ADD_PANEL_OTHER_GROUP = {
3939
getIconType: () => 'empty',
4040
order: -1, // Given an item that doesn't specify a group is assigned zero, this forces other to come after all intentionally grouped section
4141
};
42-
43-
export const ADD_PANEL_LEGACY_GROUP = {
44-
id: 'legacy',
45-
getDisplayName: () =>
46-
i18n.translate('embeddableApi.common.constants.grouping.legacy', {
47-
defaultMessage: 'Legacy',
48-
}),
49-
order: -2, // Given an item that doesn't specify a group is assigned zero, this forces it to the bottom of the list
50-
};

src/platform/plugins/shared/visualizations/public/actions/add_agg_vis_action.test.ts

Lines changed: 0 additions & 62 deletions
This file was deleted.

src/platform/plugins/shared/visualizations/public/actions/add_agg_vis_action.ts

Lines changed: 0 additions & 72 deletions
This file was deleted.

src/platform/plugins/shared/visualizations/public/actions/constants.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,4 @@
77
* License v3.0 only", or the "Server Side Public License, v 1".
88
*/
99

10-
export const ADD_AGG_VIS_ACTION_ID = 'ADD_AGG_VIS';
1110
export const ACTION_EDIT_IN_LENS = 'ACTION_EDIT_IN_LENS';

src/platform/plugins/shared/visualizations/public/actions/register_actions.ts

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99

1010
import { DataPublicPluginStart } from '@kbn/data-plugin/public';
1111
import { CONTEXT_MENU_TRIGGER } from '@kbn/embeddable-plugin/public';
12-
import { ADD_PANEL_TRIGGER, type UiActionsStart } from '@kbn/ui-actions-plugin/public';
13-
import { ACTION_EDIT_IN_LENS, ADD_AGG_VIS_ACTION_ID } from './constants';
12+
import type { UiActionsStart } from '@kbn/ui-actions-plugin/public';
13+
import { ACTION_EDIT_IN_LENS } from './constants';
1414
import { TypesStart } from '../vis_types/types_service';
1515

1616
export function registerActions(
@@ -22,9 +22,4 @@ export function registerActions(
2222
const { EditInLensAction } = await import('./edit_in_lens_action');
2323
return new EditInLensAction(data.query.timefilter.timefilter);
2424
});
25-
26-
uiActions.addTriggerActionAsync(ADD_PANEL_TRIGGER, ADD_AGG_VIS_ACTION_ID, async () => {
27-
const { AddAggVisualizationPanelAction } = await import('./add_agg_vis_action');
28-
return new AddAggVisualizationPanelAction(types);
29-
});
3025
}

test/functional/apps/dashboard/group1/create_and_add_embeddables.ts

Lines changed: 0 additions & 114 deletions
This file was deleted.

0 commit comments

Comments
 (0)