From 607a82aa86aa86da1a778698fe2233d1438f6e96 Mon Sep 17 00:00:00 2001
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sat, 29 Jul 2017 12:10:23 +0000
Subject: [PATCH] Use tabs to switch between file changes and diff of a commit.

git-svn-id: http://svn.redmine.org/redmine/trunk@16911 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 app/views/repositories/_changeset.html.erb | 23 ++++++++++++++++++++++
 app/views/repositories/revision.html.erb   |  8 --------
 2 files changed, 23 insertions(+), 8 deletions(-)

diff --git a/app/views/repositories/_changeset.html.erb b/app/views/repositories/_changeset.html.erb
index 26ca1556e..5809be241 100644
--- a/app/views/repositories/_changeset.html.erb
+++ b/app/views/repositories/_changeset.html.erb
@@ -40,3 +40,26 @@
 <% if @changeset.issues.visible.any? || User.current.allowed_to?(:manage_related_issues, @repository.project) %>
   <%= render :partial => 'related_issues' %>
 <% end %>
+
+<% if User.current.allowed_to?(:browse_repository, @repository.project) %>
+<%
+tabs = []
+tabs << { name: 'revision', label: :label_change_plural,
+          url: { :action => 'revision',
+                 :id     => @project,
+                 :repository_id => @repository.identifier_param,
+                 :path   => nil,
+                 :rev    => @changeset.identifier}
+        }
+
+tabs << { name: 'diff', label: :label_view_diff,
+          url: { :action => 'diff',
+                 :id     => @project,
+                 :repository_id => @repository.identifier_param,
+                 :path   => "",
+                 :rev    => @changeset.identifier }
+        } if action_name == 'diff' || @changeset.filechanges.any?
+%>
+
+<%= render :partial => 'common/tabs', :locals => {:tabs => tabs, :selected_tab => action_name} %>
+<% end %>
diff --git a/app/views/repositories/revision.html.erb b/app/views/repositories/revision.html.erb
index 9d6116f5f..5c215c0eb 100644
--- a/app/views/repositories/revision.html.erb
+++ b/app/views/repositories/revision.html.erb
@@ -29,7 +29,6 @@
 <%= render :partial => 'changeset' %>
 
 <% if User.current.allowed_to?(:browse_repository, @project) %>
-<h3><%= l(:label_attachment_plural) %></h3>
 <ul id="changes-legend">
 <li class="change change-A"><%= l(:label_added)    %></li>
 <li class="change change-M"><%= l(:label_modified) %></li>
@@ -38,13 +37,6 @@
 <li class="change change-D"><%= l(:label_deleted)  %></li>
 </ul>
 
-<p><%= link_to(l(:label_view_diff),
-               :action => 'diff',
-               :id     => @project,
-               :repository_id => @repository.identifier_param,
-               :path   => "",
-               :rev    => @changeset.identifier) if @changeset.filechanges.any? %></p>
-
 <div class="changeset-changes">
 <%= render_changeset_changes %>
 </div>