Skip to content

[EuiFlyout] Adjust the source order to announce EuiTitle before Close button #7223

@1Copenut

Description

@1Copenut

Describe the problem

@kyrspl && @cee-chen are making a dismiss (close) button to EuiCallout in #7156. They proposed moving the heading (title) before the dismiss button in the source order so screen reader users could hear the message heading before deciding to dismiss the callout. I seconded this approach and suggested we should update EuiModal to have this same source order.

Proposed solution

! EuiFlyout lines 413-426
! https://github.com/elastic/eui/blob/main/src/components/flyout/flyout.tsx#L413C12-L426C12

<Element ...{props}>
  {!isPushed && screenReaderDescription}
! Potentially add a prop to ignore this Close button if a header exists
! Could be something like `hasHeader` that renders the button inside `EuiHeader`  {closeButton}
  {children}
</Element>

Related issues

WCAG or Vendor Guidance (optional)

Screenshots or Trace Logs

Screen Shot 2023-09-25 at 12 45 55 PM

Metadata

Metadata

Assignees

Labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions