[wp-trac] [WordPress Trac] #29979: Twenty Fifteen: Long sidebar produces double-scrollbars

[WordPress Trac]

#29979: Twenty Fifteen: Long sidebar produces double-scrollbars
-------------------------------------+-----------------------------
 Reporter:  kraftbj                  |       Owner:
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  4.1
Component:  Bundled Theme            |     Version:  trunk
 Severity:  blocker                  |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:  ui, javascript
-------------------------------------+-----------------------------

Comment (by azaozz):

 29979.5.diff feels a bit "unnatural" at first (two page areas scrolling at
 different speeds?) but can get used to it :)

 Will need to handle couple more edge cases:
 - If the post is very short and the sidebar is quite long, the JS
 "scrolling" of the sidebar is too accelerated/too fast and is hard to see
 what is in there or stop at a particular place. If scrolling with a mouse
 wheel, it just "jumps" which is disorienting.
 - The opposite case, shorter sidebar and very long content, the sidebar
 scrolls too slow. Need to do a lot of scrolling to see the bottom.

 If this is implemented a TODO would be:
 - The default CSS for the sidebar should be `position: static` so the page
 height is set properly with one scrollbar. This will be used for small
 screens/mobile and when no-js.
 - The scroll ratio in scroll.js should have min and max limits, something
 like 0.5 to 2 seems appropriate.
 - In case the sidebar height is less than the viewport height we should
 set it to `position: fixed` (best done by adding/removing a class on
 `#sidebar` or `body`). This should be hooked in window resizing.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/29979#comment:35>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform