fix(ld-sr-live): use aria-relevant all to also announce first message

emdgroup-liquid · Oct 27, 2021 · 5c607b7 · 5c607b7
1 parent 0881c54
commit 5c607b7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/liquid/components/ld-sr-live/ld-sr-live.tsx b/src/liquid/components/ld-sr-live/ld-sr-live.tsx
@@ -46,15 +46,15 @@ export class LdSrLive {
         <span
           role="status"
           aria-live="polite"
-          aria-relevant="text"
+          aria-relevant="all"
           aria-atomic="true"
         >
           {this.info}
         </span>
         <span
           role="alert"
           aria-live="polite"
-          aria-relevant="text"
+          aria-relevant="all"
           aria-atomic="true"
         >
           {this.alert}

diff --git a/src/liquid/components/ld-sr-live/test/__snapshots__/ld-sr-live.spec.tsx.snap b/src/liquid/components/ld-sr-live/test/__snapshots__/ld-sr-live.spec.tsx.snap
@@ -4,8 +4,8 @@ exports[`ld-sr-live renders 1`] = `
 <ld-sr-live>
   <mock:shadow-root>
     <ld-sr-only>
-      <span aria-atomic="true" aria-live="polite" aria-relevant="text" role="status"></span>
-      <span aria-atomic="true" aria-live="polite" aria-relevant="text" role="alert"></span>
+      <span aria-atomic="true" aria-live="polite" aria-relevant="all" role="status"></span>
+      <span aria-atomic="true" aria-live="polite" aria-relevant="all" role="alert"></span>
     </ld-sr-only>
   </mock:shadow-root>
 </ld-sr-live>