From ade91753db557d2e1a6829f177194232da4bd221 Mon Sep 17 00:00:00 2001 From: Constance Chen Date: Thu, 2 Feb 2023 13:34:43 -0800 Subject: [PATCH] [PR feedback] Remove `role` prop customization, tweak SR text accordingly --- .../collapsible_nav.test.tsx.snap | 37 +++++--- .../collapsible_nav/collapsible_nav.tsx | 2 - .../flyout/__snapshots__/flyout.test.tsx.snap | 95 +++++-------------- src/components/flyout/flyout.test.tsx | 8 -- src/components/flyout/flyout.tsx | 38 +++----- upcoming_changelogs/6566.md | 1 + 6 files changed, 58 insertions(+), 123 deletions(-) diff --git a/src/components/collapsible_nav/__snapshots__/collapsible_nav.test.tsx.snap b/src/components/collapsible_nav/__snapshots__/collapsible_nav.test.tsx.snap index c1646d2866ae..933e1e56d3dc 100644 --- a/src/components/collapsible_nav/__snapshots__/collapsible_nav.test.tsx.snap +++ b/src/components/collapsible_nav/__snapshots__/collapsible_nav.test.tsx.snap @@ -16,6 +16,7 @@ Array [ class="euiFlyout euiCollapsibleNav emotion-euiFlyout-none-noMaxWidth-overlay-left" data-autofocus="true" id="id" + role="dialog" style="inline-size: 320px;" tabindex="0" > @@ -23,10 +24,10 @@ Array [ class="emotion-euiScreenReaderOnly" id="generated-id" > - You are in a nav. To close this nav, press Escape. Or tap/click outside the nav on the shadowed overlay to close. + You are in a modal dialog. Press Escape or tap/click outside the dialog on the shadowed overlay to close.

- - -
-
, -] -`; - exports[`EuiFlyout props sides left is rendered 1`] = ` Array [
@@ -827,7 +780,7 @@ Array [ class="emotion-euiScreenReaderOnly" id="generated-id" > - You are in a dialog. To close this dialog, press Escape. Or tap/click outside the dialog on the shadowed overlay to close. + You are in a modal dialog. Press Escape or tap/click outside the dialog on the shadowed overlay to close.