From 3250027a1d10ee66c0068c44dd4b64fde00a51e2 Mon Sep 17 00:00:00 2001
From: Kevin Buhmann <kbuhmann@vmware.com>
Date: Thu, 8 Dec 2022 10:08:33 -0600
Subject: [PATCH] feat(stepper): remove `aria-live` announcement for error
 state

BREAKING CHANGE: The stepper will no longer announce error states.
Applications can use `LiveAnnouncer` from `@angular/cdk` to handle any required announcements.
---
 .../angular/src/accordion/accordion-panel.html     |  4 ----
 .../src/accordion/stepper/stepper-panel.spec.ts    | 14 --------------
 2 files changed, 18 deletions(-)

diff --git a/projects/angular/src/accordion/accordion-panel.html b/projects/angular/src/accordion/accordion-panel.html
index eba3cee006..0e3a9ca17f 100644
--- a/projects/angular/src/accordion/accordion-panel.html
+++ b/projects/angular/src/accordion/accordion-panel.html
@@ -1,8 +1,4 @@
 <ng-container *ngIf="panel | async; let panel">
-  <div *ngIf="panel.status !== AccordionStatus.Inactive" aria-live="assertive" class="clr-sr-only">
-    <ng-container *ngIf="panel.status === AccordionStatus.Error">{{commonStrings.keys.danger}}</ng-container>
-  </div>
-
   <div [ngClass]="getPanelStateClasses(panel)">
     <div class="clr-accordion-header">
       <button
diff --git a/projects/angular/src/accordion/stepper/stepper-panel.spec.ts b/projects/angular/src/accordion/stepper/stepper-panel.spec.ts
index 04c1609386..04dbbfaf32 100644
--- a/projects/angular/src/accordion/stepper/stepper-panel.spec.ts
+++ b/projects/angular/src/accordion/stepper/stepper-panel.spec.ts
@@ -81,20 +81,6 @@ describe('ClrStep Reactive Forms', () => {
       expect(stepperPanelElement.classList.contains('clr-accordion-panel')).toBe(true);
     });
 
-    it('should show the appropriate aria-live message based on form state', () => {
-      const mockStep = new AccordionPanelModel('groupName', 0);
-      const stepperService = fixture.debugElement.query(By.directive(ClrStepperPanel)).injector.get(StepperService);
-      let liveSection: HTMLElement = fixture.nativeElement.querySelector('[aria-live="assertive"]');
-      expect(liveSection).toBe(null);
-
-      mockStep.status = AccordionStatus.Error;
-      (stepperService as MockStepperService).step.next(mockStep);
-      fixture.detectChanges();
-      liveSection = fixture.nativeElement.querySelector('[aria-live="assertive"]');
-      expect(liveSection).toBeTruthy();
-      expect(liveSection.innerText.trim()).toBe('Error');
-    });
-
     it('should show appropriate screen reader only status in button based on form state', () => {
       const mockStep = new AccordionPanelModel('groupName', 0);
       const stepperService = fixture.debugElement.query(By.directive(ClrStepperPanel)).injector.get(StepperService);