From f98b05035b429f628abe1d1844c5d07b28461488 Mon Sep 17 00:00:00 2001
From: Wladimir Coka <wlacoka@gmail.com>
Date: Mon, 28 Sep 2015 14:06:55 -0500
Subject: [PATCH] fix(dropdown): cross browser dropdown position

---
 src/uiSelectController.js |  1 +
 src/uiSelectDirective.js  | 23 +++++++++++++++++++----
 2 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/src/uiSelectController.js b/src/uiSelectController.js
index 486c245a2..fd79fb014 100644
--- a/src/uiSelectController.js
+++ b/src/uiSelectController.js
@@ -142,6 +142,7 @@ uis.controller('uiSelectCtrl',
     ctrl.itemProperty = ctrl.parserResult.itemName;
 
     ctrl.refreshItems = function (data){
+      $scope.calculateDropdownPos();
       data = data || ctrl.parserResult.source($scope);
       var selectedItems = ctrl.selected;
       //TODO should implement for single mode removeSelected
diff --git a/src/uiSelectDirective.js b/src/uiSelectDirective.js
index 0f537d196..daf9812b3 100644
--- a/src/uiSelectDirective.js
+++ b/src/uiSelectDirective.js
@@ -257,8 +257,12 @@ uis.directive('uiSelect',
             directionUpClassName = 'direction-up';
 
         // Support changing the direction of the dropdown if there isn't enough space to render it.
-        scope.$watch('$select.open', function(isOpen) {
-          if (isOpen) {
+        scope.$watch('$select.open', function() {
+          scope.calculateDropdownPos();
+        });
+
+        scope.calculateDropdownPos = function(){
+          if ($select.open) {
             dropdown = angular.element(element).querySelectorAll('.ui-select-dropdown');
             if (dropdown === null) {
               return;
@@ -269,14 +273,25 @@ uis.directive('uiSelect',
 
             // Delay positioning the dropdown until all choices have been added so its height is correct.
             $timeout(function(){
+
+              element.removeClass(directionUpClassName);
+
               var offset = uisOffset(element);
               var offsetDropdown = uisOffset(dropdown);
 
+              //https://code.google.com/p/chromium/issues/detail?id=342307#c4
+              var scrollTop = $document[0].documentElement.scrollTop || $document[0].body.scrollTop; //To make it cross browser (blink, webkit, IE, Firefox).
+
               // Determine if the direction of the dropdown needs to be changed.
-              if (offset.top + offset.height + offsetDropdown.height > $document[0].documentElement.scrollTop + $document[0].documentElement.clientHeight) {
+              if (offset.top + offset.height + offsetDropdown.height > scrollTop + $document[0].documentElement.clientHeight) {
+                //Go UP
                 dropdown[0].style.position = 'absolute';
                 dropdown[0].style.top = (offsetDropdown.height * -1) + 'px';
                 element.addClass(directionUpClassName);
+              }else{
+                //Go DOWN
+                dropdown[0].style.position = '';
+                dropdown[0].style.top = '';
               }
 
               // Display the dropdown once it has been positioned.
@@ -292,7 +307,7 @@ uis.directive('uiSelect',
               dropdown[0].style.top = '';
               element.removeClass(directionUpClassName);
           }
-        });
+        };
       };
     }
   };