Merge pull request #8 from privman/master

Do not defer focus to next tick on iOS devices
goodeggs · Jun 3, 2017 · e892797 · e892797
2 parents d8da986 + be93668
commit e892797
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 6 deletions.
diff --git a/lib/index.js b/lib/index.js
diff --git a/ng-focus-on.js b/ng-focus-on.js
@@ -17,9 +17,14 @@ app.directive('focusOn', function() {
 app.factory('focus', [
   '$rootScope', '$timeout', (function($rootScope, $timeout) {
     return function(name) {
-      return $timeout(function() {
+      var _ref;
+      if ((_ref = navigator.platform) === 'iPod' || _ref === 'iPhone' || _ref === 'iPad') {
         return $rootScope.$broadcast('focusOn', name);
-      });
+      } else {
+        return $timeout(function() {
+          return $rootScope.$broadcast('focusOn', name);
+        });
+      }
     };
   })
 ]);

diff --git a/ng-focus-on.min.js b/ng-focus-on.min.js
diff --git a/src/index.coffee b/src/index.coffee
@@ -7,6 +7,9 @@ app.directive 'focusOn', ->
 
 app.factory 'focus', ['$rootScope', '$timeout', (($rootScope, $timeout) ->
   (name) ->
-    $timeout -> # wait until next tick so that all other deferred actions happen first
+    if navigator.platform in ['iPod', 'iPhone', 'iPad'] # iOS ignores focus events that aren't triggered in a user-initiated thread
       $rootScope.$broadcast 'focusOn', name
+    else
+      $timeout -> # wait until next tick so that all other deferred actions happen first
+        $rootScope.$broadcast 'focusOn', name
 )]