Finalizes over-selection problem for taggingLabel === false being having ambiguous ctrl.activeIndex

Author: Brian Feister

dist/select.css

@@ -1,7 +1,7 @@
 /*!
  * ui-select
  * http://github.com/angular-ui/ui-select
- * Version: 0.8.3 - 2014-10-21T21:55:48.791Z
+ * Version: 0.8.3 - 2014-10-21T22:40:46.669Z
  * License: MIT
  */
 

dist/select.js

@@ -1,7 +1,7 @@
 /*!
  * ui-select
  * http://github.com/angular-ui/ui-select
- * Version: 0.8.3 - 2014-10-21T21:55:48.787Z
+ * Version: 0.8.3 - 2014-10-21T22:40:46.668Z
  * License: MIT
  */
 
@@ -320,6 +320,9 @@
 
     ctrl.isActive = function(itemScope) {
       var itemIndex = ctrl.items.indexOf(itemScope[ctrl.itemProperty]);
+      if ( ctrl.taggingLabel === false && ctrl.activeIndex === -1 ) {
+        return false;
+      }
       return ctrl.open && itemIndex === ctrl.activeIndex;
     };
 
@@ -601,6 +604,11 @@
     });
 
     _searchInput.on('keyup', function(e) {
+      if ( ! KEY.isVerticalMovement(e.which) ) {
+        $scope.$evalAsync( function () {
+          ctrl.activeIndex = ctrl.taggingLabel === false ? -1 : 0;
+        });
+      }
       // Push a "create new" item into array if there is a search string
       if ( ctrl.tagging.isActivated && ctrl.search.length > 0 ) {
 

dist/select.min.css

@@ -74,7 +74,8 @@ <h3>Simple String Tags</h3>
   <p>Selected: {{multipleDemo.colors}}</p>
   <hr>
 
-  <h3>Simple String Tags (No Labels)</h3>
+  <h3>Simple String Tags </h3>
+  <h4>(Predictive Search Model / No Labels)</h4>
   <ui-select multiple tagging tagging-label="false" ng-model="multipleDemo.colors2" theme="bootstrap" ng-disabled="disabled" style="width: 300px;">
     <ui-select-match placeholder="Select colors...">{{$item}}</ui-select-match>
     <ui-select-choices repeat="color in availableColors | filter:$select.search">

dist/select.min.js

@@ -312,6 +312,9 @@
 
     ctrl.isActive = function(itemScope) {
       var itemIndex = ctrl.items.indexOf(itemScope[ctrl.itemProperty]);
+      if ( ctrl.taggingLabel === false && ctrl.activeIndex === -1 ) {
+        return false;
+      }
       return ctrl.open && itemIndex === ctrl.activeIndex;
     };
 
@@ -593,6 +596,11 @@
     });
 
     _searchInput.on('keyup', function(e) {
+      if ( ! KEY.isVerticalMovement(e.which) ) {
+        $scope.$evalAsync( function () {
+          ctrl.activeIndex = ctrl.taggingLabel === false ? -1 : 0;
+        });
+      }
       // Push a "create new" item into array if there is a search string
       if ( ctrl.tagging.isActivated && ctrl.search.length > 0 ) {