From a09558921bafcfae91b85af563ee10144c3b7ace Mon Sep 17 00:00:00 2001 From: Valeriy Sizov Date: Mon, 2 Jul 2012 23:08:07 +0300 Subject: [PATCH] InlineDiff: base implementation --- app/assets/stylesheets/sections/commits.scss | 11 ++++ app/helpers/commits_helper.rb | 55 +++++++++++++++++++- 2 files changed, 64 insertions(+), 2 deletions(-) diff --git a/app/assets/stylesheets/sections/commits.scss b/app/assets/stylesheets/sections/commits.scss index 078709e2..acab785a 100644 --- a/app/assets/stylesheets/sections/commits.scss +++ b/app/assets/stylesheets/sections/commits.scss @@ -82,6 +82,17 @@ color:#333; font-size: 12px; font-family: 'Menlo', 'Liberation Mono', 'Consolas', 'Courier New', 'andale mono','lucida console',monospace; + .old{ + span.idiff{ + background-color:#FAA; + } + } + .new{ + span.idiff{ + background-color:#AFA; + } + } + } .diff_file_content_image { background:#eee; diff --git a/app/helpers/commits_helper.rb b/app/helpers/commits_helper.rb index 7ee85a5b..cbd2c969 100644 --- a/app/helpers/commits_helper.rb +++ b/app/helpers/commits_helper.rb @@ -35,8 +35,9 @@ module CommitsHelper line_old = 1 line_new = 1 type = nil - - lines_arr = diff_arr + + lines_arr = inline_diff diff_arr + #lines_arr = diff_arr lines_arr.each do |line| next if line.match(/^\-\-\- \/dev\/null/) next if line.match(/^\+\+\+ \/dev\/null/) @@ -45,6 +46,9 @@ module CommitsHelper full_line = html_escape(line.gsub(/\n/, '')) + full_line.gsub!("#!idiff-start!#", "") + full_line.gsub!("#!idiff-finish!#", "") + if line.match(/^@@ -/) type = "match" @@ -81,4 +85,51 @@ module CommitsHelper nil end end + + def inline_diff diff_arr + chain_of_first_symbols = "" + diff_arr.each_with_index do |line, i| + chain_of_first_symbols += line[0] + end + chain_of_first_symbols.gsub!(/[^\-\+]/, "#") + + offset = 0 + indexes = [] + while index = chain_of_first_symbols.index("#-+#", offset) + indexes << index + offset = index + 1 + end + + indexes.each do |index| + first_line = diff_arr[index+1] + second_line = diff_arr[index+2] + max_length = [first_line.size, second_line.size].max + + first_the_same_symbols = 0 + (0..max_length + 1).each do |i| + first_the_same_symbols = i - 1 + if first_line[i] != second_line[i] && i > 0 + break + end + end + first_token = first_line[0..first_the_same_symbols][1..-1] + + diff_arr[index+1].sub!(first_token, first_token + "#!idiff-start!#") + diff_arr[index+2].sub!(first_token, first_token + "#!idiff-start!#") + + last_the_same_symbols = 0 + (1..max_length + 1).each do |i| + last_the_same_symbols = -i + if first_line[-i] != second_line[-i] + break + end + end + last_the_same_symbols += 1 + last_token = first_line[last_the_same_symbols..-1] + + diff_arr[index+1].sub!(/#{last_token}$/, "#!idiff-finish!#" + last_token) + diff_arr[index+2].sub!(/#{last_token}$/, "#!idiff-finish!#" + last_token) + end + diff_arr + end end