From a6d8aa827d645796cf986e289b7311360318a21f Mon Sep 17 00:00:00 2001 From: Louis-Maxime Piton Date: Mon, 20 Mar 2023 01:45:46 +0100 Subject: [PATCH] Proposal to use `scroll-margin-top` instead of introducing padding and negative margin (#38220) --- site/assets/scss/_content.scss | 5 ----- site/assets/scss/_scrolling.scss | 5 ++++- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/site/assets/scss/_content.scss b/site/assets/scss/_content.scss index 9eca804645..3ca1dc3b2d 100644 --- a/site/assets/scss/_content.scss +++ b/site/assets/scss/_content.scss @@ -4,11 +4,6 @@ .bd-content { // Offset content from fixed navbar when jumping to headings - > :target { - padding-top: 5rem; - margin-top: -5rem; - } - > h2, > h3, > h4 { diff --git a/site/assets/scss/_scrolling.scss b/site/assets/scss/_scrolling.scss index 84db8809a7..201796003e 100644 --- a/site/assets/scss/_scrolling.scss +++ b/site/assets/scss/_scrolling.scss @@ -3,8 +3,11 @@ main { a, button, + h2, + h3, + h4, [tabindex="0"] { - scroll-margin-top: 100px; + scroll-margin-top: 80px; scroll-margin-bottom: 100px; } }