diff options
author | Gaël Poupard <ffoodd@users.noreply.github.com> | 2021-02-10 08:27:43 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-02-10 08:27:43 +0300 |
commit | 2bb42a9176e7ea361aa1ab0050da5be6a5cb56ea (patch) | |
tree | d7bf9700f7c7dd41a15bf53de8a29223319077fb /js/dist/toast.js.map | |
parent | 2ab6dbd2d696d403c6914d820a3847542d468df1 (diff) |
docs(scroll offset): improve a11y for sticky header (#33027)
Our current trick works great for headings or landmarks `:target`ed by links activation, however it still allows focus to be hidden under the sticky header.
1. Go to any docs page,
2. use a TOC link to scroll down the page,
3. then `Shift`+`Tab` to navigate backward using keyboard.
You should see on some browsers (mainly Firefox from my tests, but it used to do the same in Chrome) that focused elements isn't visible, hidden by our sticky header.
Applying `scroll-padding-top` to the `:root` solves this without any side-effect—but it requires to drop `scroll-margin-top` for anchors to prevent doubling the offset.
FWIW based on my test, Edgium and Chromium don't really care since they seem to have a new heuristic for such cases, making the focused element scroll to the center of the view.
Diffstat (limited to 'js/dist/toast.js.map')
0 files changed, 0 insertions, 0 deletions