Proposal to use `scroll-margin-top` instead of introducing padding and negative margi...

diff --git a/site/assets/scss/_content.scss b/site/assets/scss/_content.scss
index 9eca80464539173ae4fd7493cdd063864d1bdcba..3ca1dc3b2d91862d154a658fa92c1a98500cd54d 100644 (file)
--- 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 84db8809a7f973bc26172f5f144a7c7a966fa53f..201796003e7f932ad4f9311176f2b5598df704b8 100644 (file)
--- 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;
   }
 }