fix(slider): prevent focus when disabled (#131)

allianz · Dec 3, 2020 · b7ac7ef · b7ac7ef
1 parent 3c64db8
commit b7ac7ef
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/projects/ng-aquila/src/slider/slider.component.spec.ts b/projects/ng-aquila/src/slider/slider.component.spec.ts
@@ -321,6 +321,7 @@ describe('NxSliderComponent', () => {
       // value of disabled slider should not have changed
       expect(sliderInstance.value).toBe(42);
       expect(sliderNativeElement.hasAttribute('aria-disabled')).toBe(true);
+      expect(sliderNativeElement.tabIndex).toBe(-1);
       expect(sliderNativeElement.classList).toContain('nx-slider--disabled');
     });
   });

diff --git a/projects/ng-aquila/src/slider/slider.component.ts b/projects/ng-aquila/src/slider/slider.component.ts
@@ -80,7 +80,7 @@ export class NxSliderComponent implements ControlValueAccessor, AfterViewInit, O
     this._changeDetectorRef.markForCheck();
   }
   get tabindex(): number {
-    return this._tabIndex;
+    return this._disabled ? -1 : this._tabIndex;
   }
 
   private _min: number = DEFAULT_MIN;