From 57dc5f1daf692595c93c5c0b453570067020c07c Mon Sep 17 00:00:00 2001 From: Pradyun Gedam Date: Tue, 30 Jan 2024 01:33:31 +0000 Subject: [PATCH] Add an offset of `:target` to reduce back-to-top overlap --- src/furo/assets/scripts/furo.js | 2 +- src/furo/assets/styles/_scaffold.sass | 2 +- src/furo/assets/styles/content/_target.sass | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/furo/assets/scripts/furo.js b/src/furo/assets/scripts/furo.js index 3e99d8ac6..3fb3cf171 100644 --- a/src/furo/assets/scripts/furo.js +++ b/src/furo/assets/scripts/furo.js @@ -141,7 +141,7 @@ function setupScrollSpy() { navClass: "scroll-current", offset: () => { let rem = parseFloat(getComputedStyle(document.documentElement).fontSize); - return header.getBoundingClientRect().height + 0.5 * rem + 1; + return header.getBoundingClientRect().height + 2.5 * rem + 1; }, }); } diff --git a/src/furo/assets/styles/_scaffold.sass b/src/furo/assets/styles/_scaffold.sass index 39cb6c646..278b70fa1 100644 --- a/src/furo/assets/styles/_scaffold.sass +++ b/src/furo/assets/styles/_scaffold.sass @@ -406,7 +406,7 @@ article // Add a scroll margin for the content :target - scroll-margin-top: var(--header-height) + scroll-margin-top: calc(var(--header-height) + 2.5rem) // Show back-to-top below the header .back-to-top diff --git a/src/furo/assets/styles/content/_target.sass b/src/furo/assets/styles/content/_target.sass index 4de8e499b..516bbcc8b 100644 --- a/src/furo/assets/styles/content/_target.sass +++ b/src/furo/assets/styles/content/_target.sass @@ -1,13 +1,13 @@ :target - scroll-margin-top: 0.5rem + scroll-margin-top: 2.5rem @media (max-width: $full-width - $sidebar-width) :target - scroll-margin-top: calc(0.5rem + var(--header-height)) + scroll-margin-top: calc(2.5rem + var(--header-height)) // When a heading is selected section > span:target - scroll-margin-top: calc(0.8rem + var(--header-height)) + scroll-margin-top: calc(2.8rem + var(--header-height)) // Permalinks .headerlink