fix(ngAria): remove tabindex from radio buttons

Closes angular#12492
marcysutton · Oct 14, 2015 · 6a0ea9e · 6a0ea9e
1 parent 9f716dd
commit 6a0ea9e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/ngAria/aria.js b/src/ngAria/aria.js
@@ -235,7 +235,8 @@ ngAriaModule.directive('ngShow', ['$aria', function($aria) {
           }
         },
         post: function(scope, elem, attr, ngModel) {
-          var needsTabIndex = shouldAttachAttr('tabindex', 'tabindex', elem);
+          var needsTabIndex = shouldAttachAttr('tabindex', 'tabindex', elem)
+                                && !isNodeOneOf(elem, nodeBlackList);
 
           function ngAriaWatchModelValue() {
             return ngModel.$modelValue;

diff --git a/test/ngAria/ariaSpec.js b/test/ngAria/ariaSpec.js
@@ -628,6 +628,13 @@ describe('$aria', function() {
       expectAriaAttrOnEachElement(element, 'tabindex', undefined);
     });
 
+    it('should not attach to native controls via ngmodel', function() {
+      scope.$apply('val = null');
+      compileElement('<input type="radio" name="value" value="1" ng-model="val">');
+
+      expect(element.attr('tabindex')).toBeUndefined();
+    });
+
     it('should not attach to random ng-model elements', function() {
       compileElement('<div ng-model="val"></div>');
       expect(element.attr('tabindex')).toBeUndefined();