Merge pull request #2051 from zollinger/fix/ensure-highlight-visible

Jefiozie · web-flow · commit 39e938db5795 · 2017-10-07T18:02:15.000+02:00
fix(uiSelectCtrl): ensureHighlightVisible triggered with closed dropdown
diff --git a/src/uiSelectController.js b/src/uiSelectController.js
@@ -158,7 +158,7 @@ uis.controller('uiSelectCtrl',
       } else {
         $timeout(function () {
           ctrl.focusSearchInput(initSearchValue);
-          if(!ctrl.tagging.isActivated && ctrl.items.length > 1) {
+          if(!ctrl.tagging.isActivated && ctrl.items.length > 1 && ctrl.open) {
             _ensureHighlightVisible();
           }
         });

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ uis.controller('uiSelectCtrl',`
`158`	`158`	`} else {`
`159`	`159`	`$timeout(function () {`
`160`	`160`	`ctrl.focusSearchInput(initSearchValue);`
`161`		`- if(!ctrl.tagging.isActivated && ctrl.items.length > 1) {`
	`161`	`+ if(!ctrl.tagging.isActivated && ctrl.items.length > 1 && ctrl.open) {`
`162`	`162`	`_ensureHighlightVisible();`
`163`	`163`	`}`
`164`	`164`	`});`