Add scroll-margin-top to account for sticky header

Author: jpraet

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();
       }
     }
   }

web_src/less/_review.less

@@ -173,3 +173,15 @@ a.blob-excerpt:hover {
 .review-box > .segment {
   border: none !important;
 }
+
+.pull.files.diff [id] {
+  scroll-margin-top: 99px;
+
+  @media @mediaMd {
+    scroll-margin-top: 130px;
+  }
+
+  @media @mediaSm {
+    scroll-margin-top: 130px;
+  }
+}