Merge pull request #2315 from koenpunt/gh-issue-513

Added compare switch

app/assets/stylesheets/sections/commits.scss

@@ -300,3 +300,12 @@
   color: #fff;
   font-family: $monospace;
 }
+
+
+.commits-compare-switch{
+  background: url('switch_icon.png');
+  width: 16px;
+  height: 18px;
+  text-indent: -9999px;
+  display: inline-block;
+}

app/views/compare/_form.html.haml

@@ -13,12 +13,16 @@
       = text_field_tag :from, params[:from], placeholder: "master", class: "xlarge"
       = "..."
       = text_field_tag :to, params[:to], placeholder: "aa8b4ef", class: "xlarge"
+      - if params[:to] && params[:from]
+        = link_to 'switch', {from: params[:to], to: params[:from]}, {class: 'commits-compare-switch has_tooltip', title: 'Switch base of comparison'}
     - if @refs_are_same
       .alert
         %span Refs are the same
     .actions
       = submit_tag "Compare", class: "btn primary wide commits-compare-btn"
 
+
+ 
 :javascript
   $(function() {
     var availableTags = #{@project.ref_names.to_json};

features/steps/project/project_browse_commits.rb

@@ -32,8 +32,8 @@ class ProjectBrowseCommits < Spinach::FeatureSteps
   end
 
   And 'I fill compare fields with refs' do
-    fill_in "from", with: "bcf03b5de6c33f3869ef70d68cf06e679d1d7f9a"
+    fill_in "from", with: "8716fc78f3c65bbf7bcf7b574febd583bc5d2812"
-    fill_in "to",   with: "8716fc78f3c65bbf7bcf7b574febd583bc5d2812"
+    fill_in "to",   with: "bcf03b5de6c33f3869ef70d68cf06e679d1d7f9a"
     click_button "Compare"
   end