Proposal to use scroll-margin-top instead of introducing padding an…

…d negative margin (#38220)
twbs · Mar 20, 2023 · a6d8aa8 · a6d8aa8
1 parent 5d40483
commit a6d8aa8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git 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
@@ -3,8 +3,11 @@
 main {
   a,
   button,
+  h2,
+  h3,
+  h4,
   [tabindex="0"] {
-    scroll-margin-top: 100px;
+    scroll-margin-top: 80px;
     scroll-margin-bottom: 100px;
   }
 }