fix(tooltip): add & remove aria attributes #1710

IgniteUI · Sep 17, 2018 · aa7180e · aa7180e
1 parent 3a557da
commit aa7180e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/projects/igniteui-angular/src/lib/directives/toggle/toggle.directive.ts b/projects/igniteui-angular/src/lib/directives/toggle/toggle.directive.ts
@@ -137,6 +137,7 @@ export class IgxToggleDirective implements IToggleView, OnInit, OnDestroy {
      * @hidden
      */
     @HostBinding('class.igx-toggle--hidden')
+    @HostBinding('attr.aria-hidden')
     public get hiddenClass() {
         return this.collapsed;
     }

diff --git a/projects/igniteui-angular/src/lib/directives/tooltip/tooltip.directive.ts b/projects/igniteui-angular/src/lib/directives/tooltip/tooltip.directive.ts
@@ -284,13 +284,16 @@ export class IgxTooltipDirective extends IgxToggleDirective {
 
     @Input() public labelId = `${this.id}-label`;
 
-    @HostBinding('attr.aria-labelledby')
-    @Input('aria-labelledby')
-    public ariaLabelledBy = this.labelId;
+    // @HostBinding('attr.aria-live')
+    // public get ariaLive() {
+    //     return !this.collapsed? 'assertive': 'off';
+    // }
+
+    @HostBinding('attr.role')
+    public get role() {
+        return 'tooltip';
+    }
 
-    @HostBinding('attr.aria-label')
-    @Input('aria-label')
-    public ariaLabel: string | null = null;
 }
 
 @NgModule({