Merge branch 'fix-line-highlighter-link-click' into 'master'

Fix bug where clicking on the link icon on a file would result in "#Lundefined" in the URL Closes #2002 See merge request !988

Merge branch 'fix-line-highlighter-link-click' into 'master'
4998f7a0 · Robert Speicher · a84ed6eb · 5bac6455 · 4998f7a0
Commit 4998f7a0 authored Jul 18, 2015 by Robert Speicher
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

line_highlighter.js.coffee app/assets/javascripts/line_highlighter.js.coffee +1 -1

No files found.
--- a/app/assets/javascripts/line_highlighter.js.coffee
+++ b/app/assets/javascripts/line_highlighter.js.coffee
@@ -70,7 +70,7 @@ class @LineHighlighter

    @clearHighlight()

-    lineNumber = $(event.target).data('line-number')
+    lineNumber = $(event.target).closest('a').data('line-number')
    current = @hashToRange(@_hash)

    unless current[0] && event.shiftKey