fix(select): remove broken ARIA attributes

Removes ARIA attributes unnecessary for combobox role that failed to match elements not in the DOM
angular · Apr 16, 2015 · aaa6e5a · aaa6e5a
1 parent 70fc629
commit aaa6e5a
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 9 deletions.
diff --git a/src/components/select/select.js b/src/components/select/select.js
@@ -259,7 +259,6 @@ function SelectDirective($mdSelect, $mdUtil, $mdTheming, $interpolate, $compile,
       element.attr({
         'role': 'combobox',
         'id': 'select_' + $mdUtil.nextUid(),
-        'aria-haspopup': true,
         'aria-expanded': 'false',
         'aria-labelledby': labelEl.attr('id')
       });
@@ -760,7 +759,6 @@ function SelectProvider($$interimElementProvider) {
 
       function configureAria() {
         opts.selectEl.attr('aria-labelledby', opts.target.attr('id'));
-        opts.target.attr('aria-owns', opts.selectEl.attr('id'));
         opts.target.attr('aria-expanded', 'true');
       }
 

diff --git a/src/components/select/select.spec.js b/src/components/select/select.spec.js
@@ -561,13 +561,6 @@ describe('<md-select-menu>', function() {
       selectMenus.remove();
     }));
 
-    it('sets up the aria-owns attribute', inject(function($document) {
-      openSelect(el);
-      var selectMenu = $document.find('md-select-menu');
-      var selectMenuId = selectMenu.attr('id');
-      expect(selectMenuId.length).toBeTruthy();
-      expect(el.attr('aria-owns')).toBe(selectMenuId);
-    }));
     it('sets up the aria-labeledby attribute', inject(function($document) {
       openSelect(el);
       var selectId = el.attr('id');