[wp-trac] [WordPress Trac] #46371: Make sure in-page anchors are not hidden behind top bars

[WordPress Trac]

#46371: Make sure in-page anchors are not hidden behind top bars
---------------------------------------+-----------------------------------
 Reporter:  afercia                    |       Owner:  SergeyBiryukov
     Type:  defect (bug)               |      Status:  reviewing
 Priority:  normal                     |   Milestone:  Future Release
Component:  Toolbar                    |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:  ui, accessibility,
                                       |  css
---------------------------------------+-----------------------------------
Changes (by sabernhardt):

 * keywords:  has-screenshots good-first-bug has-patch 2nd-opinion => has-
     screenshots has-patch
 * focuses:  ui, accessibility => ui, accessibility, css


Comment:

 The `scroll-padding-top` value should be better within the stylesheet to
 give spacing on the admin side, too. Currently, clicking the "Skip to main
 content" link hides the Screen options and Help buttons behind the
 toolbar. (I realized that after @ryelle mentioned moving the height
 variable in #52623.)

 So [attachment:"46371.2.diff"] revises @Nyiriland's patch from #51910.

 Firefox scrolls a little extra for the Continue reading (read more) link
 targets. I also checked in Chrome and Edge (Windows 10).

 The scroll padding seems to work with most bundled themes. It does not
 have enough spacing for link targets in Twenty Sixteen because of the 21px
 pseudo-element "border" on larger screens, but it's closer. The CSS
 variable from #52623 might help with that and similar themes later.

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