summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-11-28 19:26:21 +0200
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2013-11-28 19:26:21 +0200
commit2e07865f16f96bc4706758cc5f2634574d38be37 (patch)
tree7189c0c90b1ba9b391bcbd5fa1f8ae7304a8ec3f
parent20bcbc195625e75f97a9b01c84e8d4e15af00d4c (diff)
parent68726c5bb4b2fed89e9bce9de76e383eccb0a365 (diff)
downloadgitlab-ce-2e07865f16f96bc4706758cc5f2634574d38be37.tar.gz
Merge remote-tracking branch 'origin/side-by-side'
Conflicts: CHANGELOG Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-rw-r--r--CHANGELOG1
-rw-r--r--app/views/projects/commits/_diffs.html.haml4
-rw-r--r--app/views/projects/commits/_parallel_view.html.haml2
-rw-r--r--features/project/commits/commits.feature8
-rw-r--r--features/steps/project/project_browse_commits.rb12
-rw-r--r--vendor/assets/javascripts/ace-src-noconflict/mode-diff.js2
6 files changed, 14 insertions, 15 deletions
diff --git a/CHANGELOG b/CHANGELOG
index f3911240c29..bb85d20985d 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,7 +4,6 @@ v 6.4.0
- Fixed another 500 error with submodules
- UI: More compact issues page
- Minimal password length increased to 8 symbols
- - Parallel diff view (Steven Thonus)
- Internal projects (Jason Hollingsworth)
v 6.3.0
diff --git a/app/views/projects/commits/_diffs.html.haml b/app/views/projects/commits/_diffs.html.haml
index aec7f15cdb7..2a32f56e8f0 100644
--- a/app/views/projects/commits/_diffs.html.haml
+++ b/app/views/projects/commits/_diffs.html.haml
@@ -31,9 +31,9 @@
and
%strong.cred #{@commit.stats.deletions} deletions
- if params[:view] == 'parallel'
- = link_to "Unified Diff", url_for(view: 'unified'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
+ = link_to "Inline Diff", url_for(view: 'inline'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
- else
- = link_to "Parallel Diff", url_for(view: 'parallel'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
+ = link_to "Side-by-side Diff", url_for(view: 'parallel'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
.file-stats
= render "projects/commits/diff_head", diffs: diffs
diff --git a/app/views/projects/commits/_parallel_view.html.haml b/app/views/projects/commits/_parallel_view.html.haml
index 6585bd140a1..3234e9da0ac 100644
--- a/app/views/projects/commits/_parallel_view.html.haml
+++ b/app/views/projects/commits/_parallel_view.html.haml
@@ -1,4 +1,4 @@
-/ Parallel diff view
+/ Side-by-side diff view
- old_file = get_old_file(project, @commit, diff)
- deleted_lines = {}
- added_lines = {}
diff --git a/features/project/commits/commits.feature b/features/project/commits/commits.feature
index 97113871a0a..cbe8b321507 100644
--- a/features/project/commits/commits.feature
+++ b/features/project/commits/commits.feature
@@ -14,12 +14,12 @@ Feature: Project Browse commits
Scenario: I browse commit from list
Given I click on commit link
Then I see commit info
- And I see parallel diff button
+ And I see side-by-side diff button
- Scenario: I browse commit with parallel diff view
+ Scenario: I browse commit with side-by-side diff view
Given I click on commit link
- And I click parallel diff button
- Then I see unified diff button
+ And I click side-by-side diff button
+ Then I see inline diff button
Scenario: I compare refs
Given I visit compare refs page
diff --git a/features/steps/project/project_browse_commits.rb b/features/steps/project/project_browse_commits.rb
index 17c51dc9a96..d667b58240f 100644
--- a/features/steps/project/project_browse_commits.rb
+++ b/features/steps/project/project_browse_commits.rb
@@ -89,16 +89,16 @@ class ProjectBrowseCommits < Spinach::FeatureSteps
links[1]['href'].should =~ %r{blob/cc1ba255d6c5ffdce87a357ba7ccc397a4f4026b}
end
- Given 'I click parallel diff button' do
- click_link "Parallel Diff"
+ Given 'I click side-by-side diff button' do
+ click_link "Side-by-side Diff"
end
- Then 'I see parallel diff button' do
- page.should have_content "Parallel Diff"
+ Then 'I see side-by-side diff button' do
+ page.should have_content "Side-by-side Diff"
end
- Then 'I see unified diff button' do
- page.should have_content "Unified Diff"
+ Then 'I see inline diff button' do
+ page.should have_content "Inline Diff"
end
end
diff --git a/vendor/assets/javascripts/ace-src-noconflict/mode-diff.js b/vendor/assets/javascripts/ace-src-noconflict/mode-diff.js
index 75e26cc7056..8d1e7cee096 100644
--- a/vendor/assets/javascripts/ace-src-noconflict/mode-diff.js
+++ b/vendor/assets/javascripts/ace-src-noconflict/mode-diff.js
@@ -66,7 +66,7 @@ var DiffHighlightRules = function() {
"regex": "^(?:\\*{15}|={67}|-{3}|\\+{3})$",
"token": "punctuation.definition.separator.diff",
"name": "keyword"
- }, { //diff.range.unified
+ }, { //diff.range.inline
"regex": "^(@@)(\\s*.+?\\s*)(@@)(.*)$",
"token": [
"constant",