Skip to content

Commit c2fba24

Browse files
authored
Merge pull request gocd#12300 from chadlwilson/fix-scroll-pinning
Fix console log menu scroll pinning
2 parents 4608031 + f3b34e7 commit c2fba24

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

server/src/main/webapp/WEB-INF/rails/app/assets/javascripts/build_detail_page.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@
150150
$(".content_wrapper_outer").toggleClass("full-screen");
151151
$(window).trigger($.Event("resetPinOnScroll"), [{
152152
calcRequiredScroll: function () {
153-
return $(".console-area").offset().top - $("#header").outerHeight(true) - $(".page_header").outerHeight(true);
153+
return $(".console-area").offset().top - $(".page_header").outerHeight(true);
154154
}
155155
}]);
156156
});

server/src/main/webapp/WEB-INF/rails/app/assets/javascripts/console_log_tailing.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@
8181
$(".console-area").on('consoleCompleted consoleUpdated consoleInteraction', function () {
8282
$(window).trigger($.Event("resetPinOnScroll"), [{
8383
calcRequiredScroll: function () {
84-
return $(".console-area").offset().top - $("#header").outerHeight(true) - $(".page_header").outerHeight(true);
84+
return $(".console-area").offset().top - $(".page_header").outerHeight(true);
8585
}
8686
}]);
8787
});

0 commit comments

Comments
 (0)