From d43bd55e5b20ba3d671b5e8e680d2b393314a6d1 Mon Sep 17 00:00:00 2001
From: Sam <sam.saffron@gmail.com>
Date: Tue, 20 Aug 2013 11:25:01 +1000
Subject: [PATCH] HACK: hold back on loading more topics for a bit while we are
 repositioning the page for Safari

---
 .../javascripts/discourse/views/topic_view.js | 28 +++++++++++++------
 1 file changed, 19 insertions(+), 9 deletions(-)

diff --git a/app/assets/javascripts/discourse/views/topic_view.js b/app/assets/javascripts/discourse/views/topic_view.js
index 176cf6060ab..daf6d6af59a 100644
--- a/app/assets/javascripts/discourse/views/topic_view.js
+++ b/app/assets/javascripts/discourse/views/topic_view.js
@@ -228,16 +228,22 @@ Discourse.TopicView = Discourse.View.extend(Discourse.Scrolling, {
     var info = Discourse.Eyeline.analyze(rows);
     if(!info) { return; }
 
-    // are we scrolling upwards?
-    if(info.top === 0 || info.onScreen[0] === 0 || info.bottom === 0) {
-      var $body = $('body');
-      var $elem = $(rows[0]);
-      var distToElement = $body.scrollTop() - $elem.position().top;
-      this.get('postStream').prependMore().then(function() {
-        Em.run.next(function () {
-          $('html, body').scrollTop($elem.position().top + distToElement);
+
+    // We disable scrolling of the topic while performing initial positioning
+    // This code needs to be refactored, the pipline for positioning posts is wack
+    // Be sure to test on safari as well when playing with this
+    if(!Discourse.TopicView.disableScroll) {
+      // are we scrolling upwards?
+      if(info.top === 0 || info.onScreen[0] === 0 || info.bottom === 0) {
+        var $body = $('body');
+        var $elem = $(rows[0]);
+        var distToElement = $body.scrollTop() - $elem.position().top;
+        this.get('postStream').prependMore().then(function() {
+          Em.run.next(function () {
+            $('html, body').scrollTop($elem.position().top + distToElement);
+          });
         });
-      });
+      }
     }
 
     // are we scrolling down?
@@ -247,6 +253,7 @@ Discourse.TopicView = Discourse.View.extend(Discourse.Scrolling, {
       this.get('postStream').appendMore();
     }
 
+
     // update dock
     this.updateDock(Ember.View.views[rows[info.bottom].id]);
 
@@ -352,6 +359,7 @@ Discourse.TopicView.reopenClass({
 
   // Scroll to a given post, if in the DOM. Returns whether it was in the DOM or not.
   jumpToPost: function(topicId, postNumber, avoidScrollIfPossible) {
+    this.disableScroll = true;
     Em.run.scheduleOnce('afterRender', function() {
       var rows = $('.topic-post.ready');
 
@@ -428,6 +436,8 @@ Discourse.TopicView.reopenClass({
               $contents.removeClass('highlighted');
             });
         }
+
+        setTimeout(function(){Discourse.TopicView.disableScroll = false;}, 500);
       }
     });
   }