Skip to content

[MessageStrip]: Should not be a live region #8394

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
1 task done
swendlandt opened this issue Mar 5, 2024 · 0 comments · Fixed by #8398
Closed
1 task done

[MessageStrip]: Should not be a live region #8394

swendlandt opened this issue Mar 5, 2024 · 0 comments · Fixed by #8398
Assignees
Labels
ACC bug This issue is a bug in the code TOPIC RL

Comments

@swendlandt
Copy link

Bug Description

The aria-live="assertive" should not be there, because static message strips should only have a role note and should not be live regions (this is also stated in the screen reader specification).

Affected Component

MessageStrip

Expected Behaviour

Message strip should not be a live region

Isolated Example

https://sap.github.io/ui5-webcomponents/playground/?path=/docs/main-message-strip--message-strip-overview

Steps to Reproduce

Generate a message strip in the "Dynamic" example

Log Output, Stack Trace or Screenshots

No response

Priority

Medium

UI5 Web Components Version

1.22.0

Browser

Chrome, Edge, Firefox, Safari

Operating System

No response

Additional Context

No response

Organization

SAP

Declaration

  • I’m not disclosing any internal or sensitive information.
@swendlandt swendlandt added the bug This issue is a bug in the code label Mar 5, 2024
@MapTo0 MapTo0 self-assigned this Mar 5, 2024
MapTo0 added a commit that referenced this issue Mar 6, 2024
MapTo0 added a commit that referenced this issue Mar 13, 2024
nnaydenow pushed a commit that referenced this issue Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ACC bug This issue is a bug in the code TOPIC RL
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

3 participants