From 88823f3e29385922cffab697d8d9f5a9a2ad3e42 Mon Sep 17 00:00:00 2001 From: mrsdizzie Date: Tue, 15 Sep 2020 16:43:05 -0400 Subject: [PATCH] Fix line selection on blame view (#12857) Blame and regular code view have diverge slightly so line highlighting wasn't working properly anymore. Make work again in lieu of a larger refactor of blame HTML to move it outside of source and into templates to match code view. Fixes #12836 Co-authored-by: techknowlogick --- web_src/js/index.js | 14 ++++++++++++-- web_src/less/_base.less | 2 +- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/web_src/js/index.js b/web_src/js/index.js index 98a103395..3f714874a 100644 --- a/web_src/js/index.js +++ b/web_src/js/index.js @@ -2005,14 +2005,24 @@ function initCodeView() { if ($('.code-view .lines-num').length > 0) { $(document).on('click', '.lines-num span', function (e) { const $select = $(this); - const $list = $('.code-view td.lines-code'); + let $list; + if ($('div.blame').length) { + $list = $('.code-view td.lines-code li'); + } else { + $list = $('.code-view td.lines-code'); + } selectRange($list, $list.filter(`[rel=${$select.attr('id')}]`), (e.shiftKey ? $list.filter('.active').eq(0) : null)); deSelect(); }); $(window).on('hashchange', () => { let m = window.location.hash.match(/^#(L\d+)-(L\d+)$/); - const $list = $('.code-view td.lines-code'); + let $list; + if ($('div.blame').length) { + $list = $('.code-view td.lines-code li'); + } else { + $list = $('.code-view td.lines-code'); + } let $first; if (m) { $first = $list.filter(`[rel=${m[1]}]`); diff --git a/web_src/less/_base.less b/web_src/less/_base.less index ddb65a7c2..82b403de6 100644 --- a/web_src/less/_base.less +++ b/web_src/less/_base.less @@ -1122,7 +1122,7 @@ i.icon.centerlock { width: 100%; } - .lines-code.active { + .active { background: #fffbdd !important; }