From b9ea4ce4f5e1e156a0f300463b8e1809a4e09735 Mon Sep 17 00:00:00 2001 From: Cee Chen <549407+cee-chen@users.noreply.github.com> Date: Mon, 15 May 2023 14:09:31 -0700 Subject: [PATCH] Upgrade EUI to v79.0.1 (#156925) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary `eui@77.2.2` ⏩ `eui@79.0.1` 🦴 The primary changes in this upgrade are around the deprecated `EuiLoadingContent` being removed in favor of `EuiSkeletonText`. - Most instances have been a [direct swap of usage](https://github.com/elastic/kibana/pull/156925/commits/327626a7bef0484016d09a2dc2fd62cc03a15edc), but [some replacements were a bit more opinionated](https://github.com/elastic/kibana/pull/156925/commits/e6ceb36a7504aa742a440aa7d78e79f790511cab) as I saw them as potential to take advantage of `EuiSkeletonText`'s syntactical sugar and screen reader announcements for when state switches to loaded. --- ## [`79.0.1`](https://github.com/elastic/eui/tree/v79.0.1) **Bug fixes** - Fixed broken push `EuiFlyout` behavior ([#6764](https://github.com/elastic/eui/pull/6764)) ## [`79.0.0`](https://github.com/elastic/eui/tree/v79.0.0) - Updated all `EuiSkeleton` components with new props that allow for more control over screen reader live announcements: `announceLoadingStatus`, `announceLoadedStatus`, and `ariaLiveProps` ([#6752](https://github.com/elastic/eui/pull/6752)) - Improved keyboard accessibility in `EuiPageHeader` by ensuring the right side menu items come into focus from left to right. ([#6753](https://github.com/elastic/eui/pull/6753)) **Breaking changes** - Removed deprecated `EuiLoadingContent`. Use the `EuiSkeleton` components instead. ([#6754](https://github.com/elastic/eui/pull/6754)) ## [`78.0.0`](https://github.com/elastic/eui/tree/v78.0.0) - Improved the contrast ratio of `EuiCheckbox`, `EuiRadio`, and `EuiSwitch` in their unchecked states to meet WCAG AA guidelines. ([#6729](https://github.com/elastic/eui/pull/6729)) - Added React Testing Library `*ByTestSubject` custom commands to `within()`. RTL utilities can be imported from `@elastic/eui/lib/test/rtl`. ([#6737](https://github.com/elastic/eui/pull/6737)) - Updated `EuiAvatar` to support a new letter `casing` prop that allow customizing text capitalization ([#6739](https://github.com/elastic/eui/pull/6739)) - Updated `EuiFocusTrap` to support the `gapMode` prop configuration (now defaults to `padding`) ([#6744](https://github.com/elastic/eui/pull/6744)) **Bug fixes** - Fixed inconsistency in `EuiSearchBar`'s AND/OR semantics between DSL and query string generation ([#6717](https://github.com/elastic/eui/pull/6717)) - Fixed `EuiFieldNumber`'s native browser validity detection causing extra unnecessary rerenders ([#6741](https://github.com/elastic/eui/pull/6741)) - Fixed the `scrollLock` property on `EuiFocusTrap` (and other components using `EuiFocusTrap`, such as `EuiFlyout` and `EuiModal`) to no longer block scrolling on nested portalled content, such as combobox dropdowns ([#6744](https://github.com/elastic/eui/pull/6744)) **Breaking changes** - `EuiAvatar`s with the default `user` type will now default to capitalizing all initials in uppercase ([#6739](https://github.com/elastic/eui/pull/6739)) --------- Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> --- package.json | 2 +- .../src/tooltip/skeleton.tsx | 12 ++- .../src/empty_viewer_state/index.tsx | 17 ++-- .../__snapshots__/comments.test.tsx.snap | 16 ++-- .../__snapshots__/list_header.test.tsx.snap | 16 ++-- .../__snapshots__/page_template.test.tsx.snap | 2 +- src/dev/license_checker/config.ts | 2 +- .../components/editor_content_spinner.tsx | 4 +- .../__snapshots__/format_editor.test.tsx.snap | 4 +- .../field_format_editor/format_editor.tsx | 6 +- .../field_format_editor.test.tsx.snap | 4 +- .../field_format_editor.tsx | 6 +- src/plugins/discover/public/plugin.tsx | 6 +- .../public/components/code_editor/index.tsx | 6 +- .../__snapshots__/page_template.test.tsx.snap | 6 +- .../public/finder/index.tsx | 4 +- .../__snapshots__/header.test.tsx.snap | 32 +++---- .../apps/kibana_overview/_page_header.ts | 4 +- .../log_categorization_page.tsx | 4 +- .../aiops/public/shared_lazy_components.tsx | 4 +- .../error_sampler/error_sample_detail.tsx | 8 +- .../service_overview/stats/metric_item.tsx | 4 +- .../service_groups_list/service_stat.tsx | 4 +- .../intance_details.tsx | 4 +- .../storage_details_per_service.tsx | 4 +- .../app/storage_explorer/summary_stats.tsx | 4 +- .../waterfall_with_summary/index.tsx | 4 +- .../transaction_tabs.tsx | 4 +- .../waterfall/span_flyout/index.tsx | 23 ++--- .../waterfall/transaction_flyout/index.tsx | 23 ++--- .../labs/labs_flyout.tsx | 4 +- .../templates/service_group_template.tsx | 20 ++-- .../shared/service_icons/icon_popover.tsx | 4 +- .../public/components/all_cases/table.tsx | 4 +- .../public/components/files/files_table.tsx | 4 +- .../recent_cases/loading_placeholders.tsx | 4 +- .../credentials/credentials.test.tsx | 4 +- .../components/credentials/credentials.tsx | 6 +- .../documents/organic_documents.test.tsx | 4 +- .../curation/documents/organic_documents.tsx | 96 +++++++++---------- .../api/geo_upload_wizard_async_wrapper.tsx | 4 +- .../api/index_name_form_async_wrapper.tsx | 4 +- .../components/node_attrs_details.tsx | 4 +- .../metric_anomaly/components/expression.tsx | 4 +- .../log_entry_examples_loading_indicator.tsx | 4 +- .../node_details/tabs/osquery/index.tsx | 4 +- .../layer_wizard_select.test.tsx.snap | 2 +- .../flyout_body/layer_wizard_select.tsx | 4 +- x-pack/plugins/maps/public/lazy_wrapper.tsx | 4 +- .../components/page_header/page_header.tsx | 4 +- .../expandable_section/expandable_section.tsx | 4 +- .../public/application/explorer/explorer.tsx | 8 +- .../test_models/output_loading.tsx | 6 +- .../ml/public/application/routing/router.tsx | 10 +- .../load_when_in_view/load_when_in_view.tsx | 4 +- ...squery_managed_custom_button_extension.tsx | 4 +- .../osquery/public/live_queries/index.tsx | 4 +- .../osquery/public/packs/packs_table.tsx | 4 +- .../osquery/public/results/results_table.tsx | 4 +- .../public/routes/packs/edit/index.tsx | 4 +- .../public/routes/packs/list/index.tsx | 4 +- .../osquery_action/index.tsx | 6 +- .../access_agreement_page.test.tsx | 6 +- .../access_agreement_page.tsx | 4 +- .../api_keys/api_keys_grid/api_key_flyout.tsx | 8 +- .../users/edit_user/change_password_modal.tsx | 8 +- .../cypress/e2e/timelines/flyout_button.cy.ts | 2 +- .../cti_details/threat_details_view.tsx | 4 +- .../event_details/event_details.tsx | 4 +- .../insights/simple_alert_table.tsx | 4 +- .../components/add_exception_flyout/index.tsx | 11 +-- .../empty_viewer_state.tsx | 4 +- .../edit_exception_flyout/index.tsx | 4 +- .../add_to_lists_table/index.tsx | 7 +- .../linked_to_list/index.tsx | 4 +- .../components/rules_table/rules_tables.tsx | 4 +- .../detection_engine/chart_panels/index.tsx | 10 +- .../pages/list_detail_view/index.tsx | 4 +- .../components/paginated_table/index.tsx | 4 +- .../context_menu_with_router_support.tsx | 4 +- .../components/management_empty_state.tsx | 4 +- .../view/details/components/flyout_header.tsx | 4 +- .../view/details/endpoint_details.tsx | 8 +- .../endpoint_policy_artifact_cards.tsx | 4 +- .../pages/policy/view/policy_details_form.tsx | 4 +- .../components/loading_placeholders/index.tsx | 4 +- .../public/overview/components/stat_value.tsx | 10 +- .../event_details/expandable_event.tsx | 4 +- .../timeline/tabs_content/index.tsx | 14 +-- .../policy_retention_schedule.tsx | 4 +- .../components/monitor_details_panel.tsx | 4 +- .../components/monitor_location_select.tsx | 4 +- .../common/components/monitor_status.tsx | 4 +- .../common/components/thershold_indicator.tsx | 4 +- .../monitor_test_result/status_badge.tsx | 4 +- .../components/error_started_at.tsx | 4 +- .../components/error_timeline.tsx | 4 +- .../monitor_details_last_run.tsx | 4 +- .../monitor_details_status.tsx | 4 +- .../monitor_errors/failed_tests_by_step.tsx | 4 +- .../monitor_summary/alert_actions.tsx | 4 +- .../monitor_summary/last_test_run.tsx | 9 +- .../monitor_summary/monitor_alerts.tsx | 4 +- .../monitor_details_panel_container.tsx | 4 +- .../overview/overview/overview_alerts.tsx | 4 +- .../overview_errors/overview_errors.tsx | 4 +- .../overview/overview_grid_item_loader.tsx | 4 +- .../components/settings/policy_link.tsx | 4 +- .../step_details_page/step_details_status.tsx | 4 +- .../step_objects/color_palette.tsx | 13 +-- .../step_details_page/step_page_nav.tsx | 4 +- .../step_details_page/step_title.tsx | 13 +-- .../test_run_details/components/step_info.tsx | 4 +- .../components/test_run_details_status.tsx | 4 +- .../components/test_run_details/step_tabs.tsx | 4 +- .../transform_management_section.tsx | 4 +- .../alerts_page/alerts_page_flyout_body.tsx | 10 +- .../sections/alerts_table/alerts_table.tsx | 4 +- .../deprecations_count_checkpoint.tsx | 4 +- .../overview/backup_step/cloud_backup.tsx | 4 +- .../overview/logs_step/logs_step.tsx | 11 +-- yarn.lock | 15 +-- 122 files changed, 394 insertions(+), 426 deletions(-) diff --git a/package.json b/package.json index ab936f36f4af7b..5a1e897b65be4b 100644 --- a/package.json +++ b/package.json @@ -97,7 +97,7 @@ "@elastic/datemath": "5.0.3", "@elastic/elasticsearch": "npm:@elastic/elasticsearch-canary@8.6.0-canary.3", "@elastic/ems-client": "8.4.0", - "@elastic/eui": "77.2.2", + "@elastic/eui": "79.0.1", "@elastic/filesaver": "1.1.2", "@elastic/node-crypto": "1.2.1", "@elastic/numeral": "^2.5.1", diff --git a/packages/kbn-cases-components/src/tooltip/skeleton.tsx b/packages/kbn-cases-components/src/tooltip/skeleton.tsx index f0c78e2a2e215d..9ade847de0e73b 100644 --- a/packages/kbn-cases-components/src/tooltip/skeleton.tsx +++ b/packages/kbn-cases-components/src/tooltip/skeleton.tsx @@ -7,13 +7,19 @@ */ import React from 'react'; -import { EuiFlexItem, EuiLoadingContent, EuiHorizontalRule, EuiSpacer } from '@elastic/eui'; +import { + EuiFlexItem, + EuiSkeletonTitle, + EuiSkeletonText, + EuiHorizontalRule, + EuiSpacer, +} from '@elastic/eui'; const SkeletonComponent: React.FC = () => { return ( - - + + diff --git a/packages/kbn-securitysolution-exception-list-components/src/empty_viewer_state/index.tsx b/packages/kbn-securitysolution-exception-list-components/src/empty_viewer_state/index.tsx index 993c07eaa7ce70..492a34a5df76c6 100644 --- a/packages/kbn-securitysolution-exception-list-components/src/empty_viewer_state/index.tsx +++ b/packages/kbn-securitysolution-exception-list-components/src/empty_viewer_state/index.tsx @@ -10,7 +10,7 @@ import React, { useMemo } from 'react'; import type { FC } from 'react'; import { css } from '@emotion/react'; import { - EuiLoadingContent, + EuiSkeletonText, EuiImage, EuiEmptyPrompt, EuiButton, @@ -116,13 +116,16 @@ const EmptyViewerStateComponent: FC = ({ listType, ]); - if (viewerStatus === ViewerStatus.LOADING || viewerStatus === ViewerStatus.SEARCHING) - return ; - return ( - - - + + + + + ); }; diff --git a/packages/kbn-securitysolution-exception-list-components/src/exception_item_card/comments/__snapshots__/comments.test.tsx.snap b/packages/kbn-securitysolution-exception-list-components/src/exception_item_card/comments/__snapshots__/comments.test.tsx.snap index 141ba955b4e279..92aa17170f1fa1 100644 --- a/packages/kbn-securitysolution-exception-list-components/src/exception_item_card/comments/__snapshots__/comments.test.tsx.snap +++ b/packages/kbn-securitysolution-exception-list-components/src/exception_item_card/comments/__snapshots__/comments.test.tsx.snap @@ -127,7 +127,7 @@ Object { >