Revert "FIX: remove scrolling jankiness"

This reverts commit b1a799b526.
Was not working properly on mobile...

app/assets/javascripts/discourse-common/lib/raw-handlebars.js.es6

@@ -18,15 +18,13 @@ RawHandlebars.helpers['get'] = function(context, options) {
   var firstContext =  options.contexts[0];
   var val = firstContext[context];
 
-  if (context.indexOf('controller.') === 0) {
-    context = context.slice(context.indexOf('.') + 1);
+  if (context.indexOf('controller') === 0) {
+    context = context.replace(/^controller\./, '');
   }
 
-  if (val && val.isDescriptor) {
-    return Em.get(firstContext, context);
-  }
-
-  return val === undefined ? Em.get(firstContext, context) : val;
+  if (val && val.isDescriptor) { return Em.get(firstContext, context); }
+  val = val === undefined ? Em.get(firstContext, context): val;
+  return val;
 };
 
 // adds compatability so this works with stringParams

app/assets/javascripts/discourse/components/scrolling-post-stream.js.es6

@@ -93,7 +93,7 @@ export default MountWidget.extend({
     let percent = null;
 
     const offset = offsetCalculator();
-    const topCheck = Math.ceil(windowTop + offset + 5);
+    const topCheck = Math.ceil(windowTop + offset);
 
     // uncomment to debug the eyeline
     /*

app/assets/javascripts/discourse/lib/lock-on.js.es6

@@ -1,13 +1,13 @@
 import { scrollTopFor } from 'discourse/lib/offset-calculator';
 
-// Dear traveller, you are entering a zone where we are at war with the browser.
-// The browser is insisting on positioning scrollTop per the location it was in
-// the past, we are insisting on it being where we want it to be.
-// The hack is just to keep trying over and over to position the scrollbar (up to 1 second).
+// Dear traveller, you are entering a zone where we are at war with the browser
+// the browser is insisting on positioning scrollTop per the location it was in
+// the past, we are insisting on it being where we want it to be
+// The hack is just to keep trying over and over to position the scrollbar (up to 1 minute)
 //
 // The root cause is that a "refresh" on a topic page will almost never be at the
 // same position it was in the past, the URL points to the post at the top of the
-// page, so a refresh will try to bring that post into view causing drift.
+// page, so a refresh will try to bring that post into view causing drift
 //
 // Additionally if you loaded multiple batches of posts, on refresh they will not
 // be loaded.
@@ -19,7 +19,6 @@ import { scrollTopFor } from 'discourse/lib/offset-calculator';
 // 2. give up on the scrollbar and implement it ourselves (something that will happen)
 
 const SCROLL_EVENTS = "scroll.lock-on touchmove.lock-on mousedown.lock-on wheel.lock-on DOMMouseScroll.lock-on mousewheel.lock-on keyup.lock-on";
-const SCROLL_TYPES  = ["mousedown", "mousewheel", "touchmove", "wheel"];
 
 function within(threshold, x, y) {
   return Math.abs(x-y) < threshold;
@@ -35,7 +34,8 @@ export default class LockOn {
   elementTop() {
     const selected = $(this.selector);
     if (selected && selected.offset && selected.offset()) {
-      return scrollTopFor(selected.offset().top);
+      const result = selected.offset().top;
+      return scrollTopFor(result);
     }
   }
 
@@ -54,7 +54,9 @@ export default class LockOn {
     $(window).scrollTop(previousTop);
 
     const interval = setInterval(() => {
-      const top = Math.max(0, this.elementTop());
+      let top = this.elementTop();
+      if (top < 0) { top = 0; }
+
       const scrollTop = $(window).scrollTop();
 
       if (typeof(top) === "undefined" || isNaN(top)) {
@@ -66,14 +68,20 @@ export default class LockOn {
         previousTop = top;
       }
 
-      // Commit suicide after a little while
-      if (new Date().getTime() - startedAt > 1000) {
+      // We commit suicide after 3s just to clean up
+      const nowTime = new Date().getTime();
+      if (nowTime - startedAt > 1000) {
         return this.clearLock(interval);
       }
     }, 50);
 
     $('body,html').off(SCROLL_EVENTS).on(SCROLL_EVENTS, e => {
-      if (e.which > 0 || SCROLL_TYPES.includes(e.type)) {
+      if ( e.which > 0 ||
+           e.type === "mousedown" ||
+           e.type === "mousewheel" ||
+           e.type === "touchmove" ||
+           e.type === "wheel"
+      ) {
         this.clearLock(interval);
       }
     });

app/assets/javascripts/discourse/lib/offset-calculator.js.es6

@@ -5,19 +5,34 @@ export function scrollTopFor(y) {
 export default function offsetCalculator(y) {
   const $header = $('header');
   const $container = $('.posts-wrapper');
+  const containerOffset = $container.offset();
+
+  let titleHeight = 0;
   const scrollTop = y || $(window).scrollTop();
-  const titleHeight = $('#topic-title').height() || 0;
+
+  if (!containerOffset || scrollTop < containerOffset.top) {
+    titleHeight = $('#topic-title').height() || 0;
+  }
+
   const rawWinHeight = $(window).height();
-  const expectedOffset = titleHeight - ($header.find('.contents').height() || 0);
+  const windowHeight = rawWinHeight - titleHeight;
+
+  const eyeTarget = (windowHeight / 10);
+  const headerHeight = $header.outerHeight(true);
+  const expectedOffset = titleHeight - ($header.find('.contents').height() || 0) + (eyeTarget * 2);
+  const ideal = headerHeight + ((expectedOffset < 0) ? 0 : expectedOffset);
 
   if ($container.length === 0) { return expectedOffset; }
 
-  const headerHeight = $header.outerHeight(true);
-  const ideal = headerHeight + Math.max(0, expectedOffset);
   const topPos = $container.offset().top;
+
   const docHeight = $(document).height();
-  const scrollPercent = Math.min(scrollTop / (docHeight - rawWinHeight), 1.0);
-  const inter = Math.min(headerHeight, topPos - scrollTop + ($container.height() * scrollPercent));
+  let scrollPercent = Math.min((scrollTop / (docHeight-rawWinHeight)), 1.0);
+
+  let inter = topPos - scrollTop + ($container.height() * scrollPercent);
+  if (inter < headerHeight + eyeTarget) {
+    inter = headerHeight + eyeTarget;
+  }
 
   if (inter > ideal) {
     const bottom = $('#topic-bottom').offset().top;