Commit 82d0221b authored by Annabel Dunstone's avatar Annabel Dunstone

Add line type conditional to diff line helper

parent d176f873
......@@ -67,14 +67,18 @@
line-height: $code_line_height;
font-size: $code_font_size;
&.noteable_line.old:before {
content: '-';
position: absolute;
&.noteable_line.old {
&:before {
content: '-';
position: absolute;
}
}
&.noteable_line.new:before {
content: '+';
position: absolute;
&.noteable_line.new {
&:before {
content: '+';
position: absolute;
}
}
span {
......@@ -406,7 +410,7 @@
.diff-line-num:not(.js-unfold-bottom) {
a {
&:before {
content: attr(data-linenumber);
content: attr(data-linenumber);
}
}
}
......
......@@ -40,11 +40,13 @@ module DiffHelper
(unfold) ? 'unfold js-unfold' : ''
end
def diff_line_content(line)
def diff_line_content(line, line_type = nil)
if line.blank?
"  ".html_safe
else
line[0] = ''
if line_type == 'new' || line_type == 'old'
line[0] = " "
end
line
end
end
......
......@@ -23,4 +23,4 @@
= link_text
- else
= link_to "", "##{line_code}", id: line_code, data: { linenumber: link_text }
%td.line_content{class: "noteable_line #{type} #{line_code}", data: { line_code: line_code }}= diff_line_content(line.text)
%td.line_content{class: "noteable_line #{type} #{line_code}", data: { line_code: line_code }}= diff_line_content(line.text, type)
......@@ -22,7 +22,7 @@
- else
%td.old_line.diff-line-num{data: {linenumber: raw(type == "new" ? " " : line.old_pos)}}
%td.new_line.diff-line-num{data: {linenumber: raw(type == "old" ? " " : line.new_pos)}}
%td.line_content{class: "noteable_line #{type} #{line_code}", line_code: line_code}= diff_line_content(line.text)
%td.line_content{class: "noteable_line #{type} #{line_code}", line_code: line_code}= diff_line_content(line.text, type)
- if line_code == note.line_code
= render "projects/notes/diff_notes_with_reply", notes: discussion_notes
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment