scroll view when opening with j/k, fixes stringer-rss#98. only scroll with n/p when item is offscreen, fixes stringer-rss#80

Matt Swanson · Matt Swanson · commit 330b343c5aaa · 2013-05-13T18:04:12.000-04:00
diff --git a/app.rb b/app.rb
@@ -62,6 +62,7 @@ def t(*args)
       "/js/bootstrap-min.js",
       "/js/bootstrap.file-input.js",
       "/js/mousetrap-min.js",
+      "/js/jquery-visible-min.js",
       "/js/underscore-min.js",
       "/js/backbone-min.js",
       "/js/app.js"
diff --git a/app/public/js/app.js b/app/public/js/app.js
@@ -97,6 +97,7 @@ var StoryView = Backbone.View.extend({
     if (this.model.get("open")) {
       this.$el.addClass("open");
       $(".story-lead", this.$el).fadeOut(1000);
+      window.scrollTo(0, this.$el.offset().top);
     } else {
       this.$el.removeClass("open");
       $(".story-lead", this.$el).show();
@@ -105,6 +106,7 @@ var StoryView = Backbone.View.extend({
 
   itemSelected: function() {
     this.$el.toggleClass("cursor", this.model.get("selected"));
+    if (!this.$el.visible()) window.scrollTo(0, this.$el.offset().top);
   },
 
   storyClicked: function() {
diff --git a/app/public/js/jquery-visible-min.js b/app/public/js/jquery-visible-min.js
