[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 iamtakashi):

 I've tried the patch 29979.4.patch. I like the sidebar never gets out of
 site like the admin.

 Here are the issues I've spotted with the patch.

 * When we have a shorter sidebar, there is an inline style `position:
 fixed` for `.sidebar`, but that's causing an issue when a screen is
 resized to < 955px. [[Image(https://i.cloudup.com/6zKGJhNWIG.png)]]

 * On Safari (Mac), when we have a longer sidebar, after scrolling up a few
 times, the top part of the sidebar gets hidden.
 https://cloudup.com/cChe93pk860

 * The biggest issue I see is that on iPad landscape orientation. I wish we
 can just simply disable this fix but we don't have a relaiable way to
 detect touch device for front-end, do we?

 29979.4.patch is now out of sync so here is the updated patch for easier
 testing.

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