diff options
author | Phil Hughes <me@iamphill.com> | 2017-10-03 12:44:38 +0300 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2017-10-03 12:44:38 +0300 |
commit | 7dbdc9837ccbfeeb3f980becc0dfede06149331e (patch) | |
tree | eb4daf2a5e3aa1f3f7cb95888dd2d131bbb07e73 /app/helpers | |
parent | b8e47d72637167be5a225fe20b5f893c43b9a885 (diff) | |
parent | b74c643c66fd15c95ad148231ebcf4f85283ca16 (diff) |
Merge branch 'dm-copy-parallel-diff' into 'master'
Only copy old/new code when selecting left/right side of parallel diff
See merge request gitlab-org/gitlab-ce!14561
Diffstat (limited to 'app/helpers')
-rw-r--r-- | app/helpers/diff_helper.rb | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/app/helpers/diff_helper.rb b/app/helpers/diff_helper.rb index 28f591a4e22..4e4a66e8a02 100644 --- a/app/helpers/diff_helper.rb +++ b/app/helpers/diff_helper.rb @@ -33,19 +33,21 @@ module DiffHelper end def diff_match_line(old_pos, new_pos, text: '', view: :inline, bottom: false) - content = content_tag :td, text, class: "line_content match #{view == :inline ? '' : view}" - cls = ['diff-line-num', 'unfold', 'js-unfold'] - cls << 'js-unfold-bottom' if bottom + content_line_class = %w[line_content match] + content_line_class << 'parallel' if view == :parallel + + line_num_class = %w[diff-line-num unfold js-unfold] + line_num_class << 'js-unfold-bottom' if bottom html = '' if old_pos - html << content_tag(:td, '...', class: cls + ['old_line'], data: { linenumber: old_pos }) - html << content unless view == :inline + html << content_tag(:td, '...', class: [*line_num_class, 'old_line'], data: { linenumber: old_pos }) + html << content_tag(:td, text, class: [*content_line_class, 'left-side']) if view == :parallel end if new_pos - html << content_tag(:td, '...', class: cls + ['new_line'], data: { linenumber: new_pos }) - html << content + html << content_tag(:td, '...', class: [*line_num_class, 'new_line'], data: { linenumber: new_pos }) + html << content_tag(:td, text, class: [*content_line_class, ('right-side' if view == :parallel)]) end html.html_safe |