Commits on Sep 28, 2018

1. Fix overscroll issues locking scroll up on long pages. …

   This, I think, fixes #8603.
   
   I wasn't able to reproduce the exact issue, but I was able to reproduce a different one which I think is the same minus some browser behavior differences.
   
   Essentially, when you don't scroll the `body` but instead scroll an element, once you've scrolled to the end of that element, pause, and then try to scroll further, browsers now try to scroll the _parent_ element, in this case `body`. But because that isn't scrollable, some browsers apply an "overscroll bounce", others just prevent scrolling, even upwards.
   
   Steps to reproduce:
   
   1. Have a page with enough content to create a scrollbar.
   2. Scroll to the very end of that content.
   3. Hold a brief pause in scrolling.
   4. Now in one motion first scroll downwards and then immediately start scrolling upwards.
   
   Observe that you're not scrolling anywhere, or you're invoking overscroll bounces.
   
   Step 4 is crucial — it is BECAUSE we start by scrolling _downwards_ that the browser decides to try and scroll the body element instead of the element we mean to scroll (`.edit-post-layout__content`), and because the body can't be scrolled, this is causing the issue.
   
   This PR fixes that.

   

   jasmussen committed Sep 28, 2018
   Configuration menu

   • View commit details
   • Copy full SHA for a14e50a
   • Browse repository at this point

   Copy the full SHA

   a14e50a View commit details

   Browse the repository at this point in the history