Add support for tabindex on focusser input

This change adds the ability to specify tabindex on the ui-select
element and have it passed to the focusser input.  The tabindex is
removed from the ui-select div after it is applied to the focusser
input.

Author: Cory Stevens

src/select.js

@@ -624,6 +624,16 @@
 
         //Idea from: https://github.com/ivaynberg/select2/blob/79b5bf6db918d7560bdd959109b7bcfb47edaf43/select2.js#L1954
         var focusser = angular.element("<input ng-disabled='$select.disabled' class='ui-select-focusser ui-select-offscreen' type='text' aria-haspopup='true' role='button' />");
+
+        if(attrs.tabindex){
+          //tabindex might be an expression, wait until it contains the actual value before we set the focusser tabindex
+          attrs.$observe('tabindex', function(value) {
+            focusser.attr("tabindex", value);
+            //Remove the tabindex on the parent so that it is not focusable
+            element.removeAttr("tabindex");
+          });
+        }
+
         $compile(focusser)(scope);
         $select.focusser = focusser;
 

test/select.spec.js

@@ -66,6 +66,7 @@ describe('ui-select tests', function() {
       if (attrs.disabled !== undefined) { attrsHtml += ' ng-disabled="' + attrs.disabled + '"'; }
       if (attrs.required !== undefined) { attrsHtml += ' ng-required="' + attrs.required + '"'; }
       if (attrs.theme !== undefined) { attrsHtml += ' theme="' + attrs.theme + '"'; }
+      if (attrs.tabindex !== undefined) { attrsHtml += ' tabindex="' + attrs.tabindex + '"'; }
     }
 
     return compileTemplate(
@@ -204,7 +205,28 @@ describe('ui-select tests', function() {
 
     el.find(".ui-select-toggle").click();
     expect($select.open).toEqual(false);
+  });
+
+  it('should pass tabindex to focusser', function() {
+    var el = createUiSelect({tabindex: 5});
+
+    expect($(el).find('.ui-select-focusser').attr('tabindex')).toEqual('5');
+    expect($(el).attr('tabindex')).toEqual(undefined);
+  });
+
+  it('should pass tabindex to focusser when tabindex is an expression', function() {
+    scope.tabValue = 22;
+    var el = createUiSelect({tabindex: '{{tabValue + 10}}'});
+
+    expect($(el).find('.ui-select-focusser').attr('tabindex')).toEqual('32');
+    expect($(el).attr('tabindex')).toEqual(undefined);
+  });
+
+  it('should not give focusser a tabindex when ui-select does not have one', function() {
+    var el = createUiSelect();
 
+    expect($(el).find('.ui-select-focusser').attr('tabindex')).toEqual(undefined);
+    expect($(el).attr('tabindex')).toEqual(undefined);
   });
 
   it('should be disabled if the attribute says so', function() {