Fix: appendToBody with jQuery < 1.10.0; stealing focus #944

.relpaceWith in jQuery (version less then 1.10) doesn't work properly
and as so, ui-select with 'appendToBody=true' doesn't display selected
value.
Use .after() and .remove() instead of .replaceWith() to avoid this
issue.

fix for #944: Don't restore focus if it was moved out of select (irregardles of target
element). Otherwise it looks too confusing when clicking on links or any
other control that is not an input/button/textarea.

Author: AlexFess

src/uiSelectDirective.js

@@ -190,17 +190,7 @@ uis.directive('uiSelect',
           }
 
           if (!contains && !$select.clickTriggeredSelect) {
-            var skipFocusser;
-            if (!$select.skipFocusser) {
-              //Will lose focus only with certain targets
-              var focusableControls = ['input','button','textarea','select'];
-              var targetController = angular.element(e.target).controller('uiSelect'); //To check if target is other ui-select
-              skipFocusser = targetController && targetController !== $select; //To check if target is other ui-select
-              if (!skipFocusser) skipFocusser =  ~focusableControls.indexOf(e.target.tagName.toLowerCase()); //Check if target is input, button or textarea
-            } else {
-              skipFocusser = true;
-            }
-            $select.close(skipFocusser);
+            $select.close(true);
             scope.$digest();
           }
           $select.clickTriggeredSelect = false;
@@ -298,7 +288,8 @@ uis.directive('uiSelect',
           }
 
           // Move the dropdown element back to its original location in the DOM
-          placeholder.replaceWith(element);
+          placeholder.after(element);
+          placeholder.remove();
           placeholder = null;
 
           element[0].style.position = '';