Possible fix for angular-ui#1355.

Dondi Dionisio · Dondi Dionisio · commit 5a47d4a90abb · 2016-03-23T14:16:29.000-07:00
diff --git a/src/uiSelectChoicesDirective.js b/src/uiSelectChoicesDirective.js
@@ -44,8 +44,7 @@ uis.directive('uiSelectChoices',
           throw uiSelectMinErr('rows', "Expected 1 .ui-select-choices-row but got '{0}'.", choices.length);
         }
 
-        choices.attr('ng-repeat', $select.parserResult.repeatExpression(groupByExp))
-            .attr('ng-if', '$select.open'); //Prevent unnecessary watches when dropdown is closed
+        choices.attr('ng-repeat', $select.parserResult.repeatExpression(groupByExp));
         if ($window.document.addEventListener) {  //crude way to exclude IE8, specifically, which also cannot capture events
           choices.attr('ng-mouseenter', '$select.setActiveItem('+$select.parserResult.itemName +')')
               .attr('ng-click', '$select.select(' + $select.parserResult.itemName + ',$select.skipFocusser,$event)');

Original file line number	Diff line number	Diff line change
`@@ -44,8 +44,7 @@ uis.directive('uiSelectChoices',`
`44`	`44`	`throw uiSelectMinErr('rows', "Expected 1 .ui-select-choices-row but got '{0}'.", choices.length);`
`45`	`45`	`}`
`46`	`46`
`47`		`- choices.attr('ng-repeat', $select.parserResult.repeatExpression(groupByExp))`
`48`		`- .attr('ng-if', '$select.open'); //Prevent unnecessary watches when dropdown is closed`
	`47`	`+ choices.attr('ng-repeat', $select.parserResult.repeatExpression(groupByExp));`
`49`	`48`	`if ($window.document.addEventListener) { //crude way to exclude IE8, specifically, which also cannot capture events`
`50`	`49`	`choices.attr('ng-mouseenter', '$select.setActiveItem('+$select.parserResult.itemName +')')`
`51`	`50`	`.attr('ng-click', '$select.select(' + $select.parserResult.itemName + ',$select.skipFocusser,$event)');`