Skip to content

Commit 6868a63

Browse files
jughostakibanamachineelasticmachine
authored andcommitted
[Discover] Remove the legacy table (elastic#201254)
- Closes elastic#167582 ## Summary This PR removes the code related to the legacy doc table and 2 Advanced Settings: `doc_table:legacy` and `truncate:maxHeight`. The legacy table in Discover was replaced by the new data grid in v8.3. The `doc_table:legacy` Advanced Setting was added to let users switch back to the legacy table if necessary. The removal of the setting and the legacy table entirely would allow us to reduce bundle size, maintenance burden, and code complexity. Also the legacy table does not support many new features which were added to the grid only (e.g. comparing selected documents, context-aware UI based on current solution project, column resizing, bulk row selection, copy actions, new doc viewer flyout, and more). Since v8.15 `doc_table:legacy` is marked as deprecated on Advanced Settings page via elastic#179899 Since v8.16 `truncate:maxHeight` is marked as deprecated too via elastic#183736 The removal of these 2 settings and the associated code is planned for v9. ### Checklist - [x] [Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html) was added for features that require explanation or tutorials - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The `release_note:breaking` label should be applied in these situations. --------- Co-authored-by: kibanamachine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
1 parent 607d3fa commit 6868a63

File tree

128 files changed

+234
-7875
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

128 files changed

+234
-7875
lines changed

.buildkite/ftr_platform_stateful_configs.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ enabled:
6464
- test/functional/apps/dashboard/group5/config.ts
6565
- test/functional/apps/dashboard/group6/config.ts
6666
- test/functional/apps/discover/ccs_compatibility/config.ts
67-
- test/functional/apps/discover/classic/config.ts
6867
- test/functional/apps/discover/embeddable/config.ts
6968
- test/functional/apps/discover/esql/config.ts
7069
- test/functional/apps/discover/group1/config.ts

docs/management/advanced-options.asciidoc

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -208,10 +208,6 @@ The default refresh interval for the time filter. Example:
208208
[[timepicker-timedefaults]]`timepicker:timeDefaults`::
209209
The default selection in the time filter.
210210

211-
[[truncate-maxheight]]`truncate:maxHeight`::
212-
deprecated:[8.16.0]The maximum height that a cell occupies in a table. Set to 0 to disable
213-
truncation.
214-
215211
[[enableESQL]]`enableESQL`::
216212
This setting enables ES|QL in Kibana.
217213

@@ -340,14 +336,6 @@ Hides the "Time" column in *Discover* and in all saved searches on dashboards.
340336
Highlights results in *Discover* and saved searches on dashboards. Highlighting
341337
slows requests when working on big documents.
342338

343-
[[doctable-legacy]]`doc_table:legacy`::
344-
deprecated:[8.15.0] Controls the way the document table looks and works.
345-
To use the new *Document Explorer* instead of the classic view, turn off this option.
346-
The *Document Explorer* offers better data sorting, resizable columns, and a full screen view.
347-
348-
[[truncate-max-height]]`truncate:maxHeight`::
349-
The maximum height that a cell in a table can occupy. To disable truncation, set to 0.
350-
351339

352340
[float]
353341
[[kibana-ml-settings]]

packages/kbn-discover-utils/index.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ export {
1414
DEFAULT_ALLOWED_LOGS_BASE_PATTERNS,
1515
DEFAULT_COLUMNS_SETTING,
1616
DOC_HIDE_TIME_COLUMN_SETTING,
17-
DOC_TABLE_LEGACY,
1817
FIELDS_LIMIT_SETTING,
1918
HIDE_ANNOUNCEMENTS,
2019
MAX_DOC_FIELDS_DISPLAYED,
@@ -28,8 +27,6 @@ export {
2827
SHOW_FIELD_STATISTICS,
2928
SHOW_MULTIFIELDS,
3029
SORT_DEFAULT_ORDER_SETTING,
31-
TRUNCATE_MAX_HEIGHT,
32-
TRUNCATE_MAX_HEIGHT_DEFAULT_VALUE,
3330
IgnoredReason,
3431
buildDataTableRecord,
3532
buildDataTableRecordList,
@@ -45,7 +42,6 @@ export {
4542
getMessageFieldWithFallbacks,
4643
getShouldShowFieldHandler,
4744
isNestedFieldParent,
48-
isLegacyTableEnabled,
4945
usePager,
5046
calcFieldCounts,
5147
getLogLevelColor,

packages/kbn-discover-utils/src/constants.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ export const CONTEXT_STEP_SETTING = 'context:step';
1212
export const CONTEXT_TIE_BREAKER_FIELDS_SETTING = 'context:tieBreakerFields';
1313
export const DEFAULT_COLUMNS_SETTING = 'defaultColumns';
1414
export const DOC_HIDE_TIME_COLUMN_SETTING = 'doc_table:hideTimeColumn';
15-
export const DOC_TABLE_LEGACY = 'doc_table:legacy';
1615
export const FIELDS_LIMIT_SETTING = 'fields:popularLimit';
1716
export const HIDE_ANNOUNCEMENTS = 'hideAnnouncements';
1817
export const MAX_DOC_FIELDS_DISPLAYED = 'discover:maxDocFieldsDisplayed';
@@ -26,5 +25,3 @@ export const SEARCH_ON_PAGE_LOAD_SETTING = 'discover:searchOnPageLoad';
2625
export const SHOW_FIELD_STATISTICS = 'discover:showFieldStatistics';
2726
export const SHOW_MULTIFIELDS = 'discover:showMultiFields';
2827
export const SORT_DEFAULT_ORDER_SETTING = 'discover:sort:defaultOrder';
29-
export const TRUNCATE_MAX_HEIGHT = 'truncate:maxHeight';
30-
export const TRUNCATE_MAX_HEIGHT_DEFAULT_VALUE = 115;

packages/kbn-discover-utils/src/utils/build_data_record.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import type { DataTableRecord, EsHitRecord } from '../types';
1717
import { getDocId } from './get_doc_id';
1818

1919
/**
20-
* Build a record for data table, explorer + classic one
20+
* Build a record for data grid
2121
* @param doc the document returned from Elasticsearch
2222
* @param dataView this current data view
2323
* @param isAnchor determines if the given doc is the anchor doc when viewing surrounding documents

packages/kbn-discover-utils/src/utils/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,4 @@ export * from './nested_fields';
2121
export * from './get_field_value';
2222
export * from './calc_field_counts';
2323
export * from './get_visible_columns';
24-
export { isLegacyTableEnabled } from './is_legacy_table_enabled';
2524
export { DiscoverFlyouts, dismissAllFlyoutsExceptFor, dismissFlyouts } from './dismiss_flyouts';

packages/kbn-discover-utils/src/utils/is_legacy_table_enabled.ts

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

packages/kbn-discover-utils/tsconfig.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
"@kbn/field-formats-plugin",
2525
"@kbn/field-types",
2626
"@kbn/i18n",
27-
"@kbn/core-ui-settings-browser",
2827
"@kbn/expressions-plugin",
2928
"@kbn/logs-data-access-plugin",
3029
"@kbn/i18n-react",

packages/kbn-management/settings/setting_ids/index.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,6 @@ export const DISCOVER_SHOW_MULTI_FIELDS_ID = 'discover:showMultiFields';
8787
export const DISCOVER_SORT_DEFAULT_ORDER_ID = 'discover:sort:defaultOrder';
8888
export const DOC_TABLE_HIDE_TIME_COLUMNS_ID = 'doc_table:hideTimeColumn';
8989
export const DOC_TABLE_HIGHLIGHT_ID = 'doc_table:highlight';
90-
export const DOC_TABLE_LEGACY_ID = 'doc_table:legacy';
91-
export const TRUNCATE_MAX_HEIGHT_ID = 'truncate:maxHeight';
9290

9391
// Machine learning settings
9492
export const ML_ANOMALY_DETECTION_RESULTS_ENABLE_TIME_DEFAULTS_ID =

packages/kbn-unified-doc-viewer/README.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22

33
This package contains components and services for the unified doc viewer component.
44

5-
Discover (Classic view → Expanded document)
6-
7-
![image](https://github.com/elastic/kibana/assets/1178348/a0a360bf-2697-4427-a32e-c728f06f5a7e)
8-
95
Discover (Document explorer → Toggle dialog with details)
106

117
![image](https://github.com/elastic/kibana/assets/1178348/c9c11587-c53f-4bcd-8d48-aaceb64981ea)

0 commit comments

Comments
 (0)