Do not loop keyboard nav on content list.

TryGhost · Aug 7, 2014 · b7b797e · b7b797e
1 parent 4b83dfd
commit b7b797e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/core/client/routes/posts.js b/core/client/routes/posts.js
@@ -44,12 +44,13 @@ var PostsRoute = Ember.Route.extend(SimpleAuth.AuthenticatedRouteMixin, Shortcut
 
         newPosition = posts.indexOf(currentPost) + step;
 
-        //Make sure we're inbounds
+        // if we are on the first or last item
+        // just do nothing (desired behavior is to not
+        // loop around)
         if (newPosition >= length) {
-            newPosition = 0;
-        }
-        else if (newPosition < 0) {
-            newPosition = length - 1;
+            return;
+        } else if (newPosition < 0) {
+            return;
         }
         this.transitionTo('posts.post', posts.objectAt(newPosition));
     },