Download as
Browse Code »

Commit 2e07865f16f96bc4706758cc5f2634574d38be37

Authored by Dmitriy Zaporozhets
2 parents 20bcbc19 68726c5b
Exists in master and in 4 other branches 6-8-stable, 7-0-stable, 7-0-stable-spb, spb-stable

Merge remote-tracking branch 'origin/side-by-side' 

Conflicts:
	CHANGELOG

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Showing 6 changed files with 14 additions and 15 deletions   Show diff stats
CHANGELOG
  Diff comments   View file @2e07865
... ... @@ -4,7 +4,6 @@ v 6.4.0
4 4 - Fixed another 500 error with submodules
5 5 - UI: More compact issues page
6 6 - Minimal password length increased to 8 symbols
7   - - Parallel diff view (Steven Thonus)
8 7 - Internal projects (Jason Hollingsworth)
9 8  
10 9 v 6.3.0
... ...
app/views/projects/commits/_diffs.html.haml
  Diff comments   View file @2e07865
... ... @@ -31,9 +31,9 @@
31 31 and
32 32 %strong.cred #{@commit.stats.deletions} deletions
33 33 - if params[:view] == 'parallel'
34   - = link_to "Unified Diff", url_for(view: 'unified'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
  34 + = link_to "Inline Diff", url_for(view: 'inline'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
35 35 - else
36   - = link_to "Parallel Diff", url_for(view: 'parallel'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
  36 + = link_to "Side-by-side Diff", url_for(view: 'parallel'), {id: "commit-diff-viewtype", class: 'btn btn-tiny pull-right'}
37 37 .file-stats
38 38 = render "projects/commits/diff_head", diffs: diffs
39 39  
... ...
app/views/projects/commits/_parallel_view.html.haml
  Diff comments   View file @2e07865
1   -/ Parallel diff view
  1 +/ Side-by-side diff view
2 2 - old_file = get_old_file(project, @commit, diff)
3 3 - deleted_lines = {}
4 4 - added_lines = {}
... ...
features/project/commits/commits.feature
  Diff comments   View file @2e07865
... ... @@ -14,12 +14,12 @@ Feature: Project Browse commits
14 14 Scenario: I browse commit from list
15 15 Given I click on commit link
16 16 Then I see commit info
17   - And I see parallel diff button
  17 + And I see side-by-side diff button
18 18  
19   - Scenario: I browse commit with parallel diff view
  19 + Scenario: I browse commit with side-by-side diff view
20 20 Given I click on commit link
21   - And I click parallel diff button
22   - Then I see unified diff button
  21 + And I click side-by-side diff button
  22 + Then I see inline diff button
23 23  
24 24 Scenario: I compare refs
25 25 Given I visit compare refs page
... ...
features/steps/project/project_browse_commits.rb
  Diff comments   View file @2e07865
... ... @@ -89,16 +89,16 @@ class ProjectBrowseCommits &lt; Spinach::FeatureSteps
89 89 links[1]['href'].should =~ %r{blob/cc1ba255d6c5ffdce87a357ba7ccc397a4f4026b}
90 90 end
91 91  
92   - Given 'I click parallel diff button' do
93   - click_link "Parallel Diff"
  92 + Given 'I click side-by-side diff button' do
  93 + click_link "Side-by-side Diff"
94 94 end
95 95  
96   - Then 'I see parallel diff button' do
97   - page.should have_content "Parallel Diff"
  96 + Then 'I see side-by-side diff button' do
  97 + page.should have_content "Side-by-side Diff"
98 98 end
99 99  
100   - Then 'I see unified diff button' do
101   - page.should have_content "Unified Diff"
  100 + Then 'I see inline diff button' do
  101 + page.should have_content "Inline Diff"
102 102 end
103 103  
104 104 end
... ...
vendor/assets/javascripts/ace-src-noconflict/mode-diff.js
  Diff comments   View file @2e07865
... ... @@ -66,7 +66,7 @@ var DiffHighlightRules = function() {
66 66 "regex": "^(?:\\*{15}|={67}|-{3}|\\+{3})$",
67 67 "token": "punctuation.definition.separator.diff",
68 68 "name": "keyword"
69   - }, { //diff.range.unified
  69 + }, { //diff.range.inline
70 70 "regex": "^(@@)(\\s*.+?\\s*)(@@)(.*)$",
71 71 "token": [
72 72 "constant",
... ...