Commit 8e4b8499 authored by Bryce Johnson's avatar Bryce Johnson

Remove colons in dropdowns, use icon helper, and fix 'Show original' link.

(Also removes duplicate changelog entry.)
parent 98b3d6ce
...@@ -175,7 +175,6 @@ v 8.12.0 (unreleased) ...@@ -175,7 +175,6 @@ v 8.12.0 (unreleased)
- Fix Gitlab::Popen.popen thread-safety issue - Fix Gitlab::Popen.popen thread-safety issue
- Add specs to removing project (Katarzyna Kobierska Ula Budziszewska) - Add specs to removing project (Katarzyna Kobierska Ula Budziszewska)
- Clean environment variables when running git hooks - Clean environment variables when running git hooks
- Add UX improvements for merge request version diffs
- Fix Import/Export issues importing protected branches and some specific models - Fix Import/Export issues importing protected branches and some specific models
- Fix non-master branch readme display in tree view - Fix non-master branch readme display in tree view
- Add UX improvements for merge request version diffs - Add UX improvements for merge request version diffs
......
...@@ -12,9 +12,9 @@ ...@@ -12,9 +12,9 @@
%span.caret %span.caret
%ul.dropdown-menu.dropdown-menu-selectable %ul.dropdown-menu.dropdown-menu-selectable
.dropdown-title .dropdown-title
%span Version: %span Version
%button.dropdown-title-button.dropdown-menu-close %button.dropdown-title-button.dropdown-menu-close
%i.fa.fa-times.dropdown-menu-close-icon = icon('times', class: 'dropdown-menu-close-icon')
- @merge_request_diffs.each do |merge_request_diff| - @merge_request_diffs.each do |merge_request_diff|
%li %li
= link_to merge_request_version_path(@project, @merge_request, merge_request_diff), class: ('is-active' if merge_request_diff == @merge_request_diff) do = link_to merge_request_version_path(@project, @merge_request, merge_request_diff), class: ('is-active' if merge_request_diff == @merge_request_diff) do
...@@ -40,9 +40,9 @@ ...@@ -40,9 +40,9 @@
%span.caret %span.caret
%ul.dropdown-menu.dropdown-menu-selectable %ul.dropdown-menu.dropdown-menu-selectable
.dropdown-title .dropdown-title
%span Compared with: %span Compared with
%button.dropdown-title-button.dropdown-menu-close %button.dropdown-title-button.dropdown-menu-close
%i.fa.fa-times.dropdown-menu-close-icon = icon('times', class: 'dropdown-menu-close-icon')
- @comparable_diffs.each do |merge_request_diff| - @comparable_diffs.each do |merge_request_diff|
%li %li
= link_to merge_request_version_path(@project, @merge_request, @merge_request_diff, merge_request_diff.head_commit_sha), class: ('is-active' if merge_request_diff == @start_version) do = link_to merge_request_version_path(@project, @merge_request, @merge_request_diff, merge_request_diff.head_commit_sha), class: ('is-active' if merge_request_diff == @start_version) do
...@@ -67,4 +67,4 @@ ...@@ -67,4 +67,4 @@
Comments are disabled because you're comparing two versions of this merge request. Comments are disabled because you're comparing two versions of this merge request.
- else - else
Comments are disabled because you're viewing an old version of this merge request. Comments are disabled because you're viewing an old version of this merge request.
= link_to 'Show latest version', merge_request_version_path(@project, @merge_request, @merge_request_diff), class: 'btn btn-sm' = link_to 'Show latest version', diffs_namespace_project_merge_request_path(@project.namespace, @project, @merge_request), class: 'btn btn-sm'
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment