Add scroll-margin-top to account for sticky header (#16269)

Fixes #16263 Co-authored-by: zeripath <art27@cantab.net>
go-gitea · Jun 27, 2021 · eee03ae · eee03ae
1 parent f533b5d
commit eee03ae
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/web_src/js/index.js b/web_src/js/index.js
@@ -1343,7 +1343,7 @@ function initPullRequestReview() {
         $(`#code-comments-${id}`).removeClass('hide');
         $(`#code-preview-${id}`).removeClass('hide');
         $(`#hide-outdated-${id}`).removeClass('hide');
-        $(window).scrollTop(commentDiv.offset().top);
+        commentDiv[0].scrollIntoView();
       }
     }
   }

diff --git a/web_src/less/_review.less b/web_src/less/_review.less
@@ -173,3 +173,11 @@ a.blob-excerpt:hover {
 .review-box > .segment {
   border: none !important;
 }
+
+.pull.files.diff [id] {
+  scroll-margin-top: 99px;
+
+  @media @mediaMdAndDown {
+    scroll-margin-top: 130px;
+  }
+}