diff options
author | Mike Greiling <mike@pixelcog.com> | 2017-01-25 15:52:12 -0600 |
---|---|---|
committer | Mike Greiling <mike@pixelcog.com> | 2017-01-25 16:05:52 -0600 |
commit | 5fbce6832826bdea9be8d305b0dc28a533d010ae (patch) | |
tree | c2903411f77275696a320a4698c88965d6b88f58 /app/assets/javascripts/line_highlighter.js | |
parent | b74f2faa591624e95ed1194558f9a5d0feb72a54 (diff) | |
download | gitlab-ce-5fbce6832826bdea9be8d305b0dc28a533d010ae.tar.gz |
prevent LineHighlighter from updating scroll position27254-clicking-line-anchor-multiple-times-causes-scroll-position-to-over-correct
Diffstat (limited to 'app/assets/javascripts/line_highlighter.js')
-rw-r--r-- | app/assets/javascripts/line_highlighter.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/assets/javascripts/line_highlighter.js b/app/assets/javascripts/line_highlighter.js index 4620715a521..2f147704c22 100644 --- a/app/assets/javascripts/line_highlighter.js +++ b/app/assets/javascripts/line_highlighter.js @@ -74,8 +74,9 @@ // If not done this way, the line number anchor will sometimes keep its // active state even when the event is cancelled, resulting in an ugly border // around the link and/or a persisted underline text decoration. - return $('#blob-content-holder').on('click', 'a[data-line-number]', function(event) { - return event.preventDefault(); + $('#blob-content-holder').on('click', 'a[data-line-number]', function(event) { + event.preventDefault(); + event.stopPropagation(); }); }; |