diff options
| author | Rémy Coutable <remy@rymai.me> | 2016-09-28 14:59:51 +0000 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2016-09-28 14:59:51 +0000 |
| commit | 7d79a943529aad64005b7a95bee1b8a32709ec89 (patch) | |
| tree | 4b529988a49ef63a2ebf636829059294c158045f | |
| parent | 76039e0188bd64ee6c9ce3427b5a4725a3ed8e64 (diff) | |
| parent | d238ff2657c8c9a76353367c672e3d592188c42e (diff) | |
| download | gitlab-ce-7d79a943529aad64005b7a95bee1b8a32709ec89.tar.gz | |
Merge branch 'dz-improve-mr-versions' into 'master'
Fix duplicate master entries in the mr versions dropdown
## What does this MR do?
Fixes bug when "master" was duplicated per each mr version in the dropdown
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/22519, Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/22715
See merge request !6567
| -rw-r--r-- | CHANGELOG | 1 | ||||
| -rw-r--r-- | app/views/projects/merge_requests/show/_versions.html.haml | 10 |
2 files changed, 6 insertions, 5 deletions
diff --git a/CHANGELOG b/CHANGELOG index 69b5ee7124e..f92fcff951a 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -23,6 +23,7 @@ v 8.12.2 (unreleased) - Fix an issue with the "Commits" section of the cycle analytics summary. !6513 - Fix errors importing project feature and milestone models using GitLab project import - Make JWT messages Docker-compatible + - Fix duplicate branch entry in the merge request version compare dropdown v 8.12.1 - Fix a memory leak in HTML::Pipeline::SanitizationFilter::WHITELIST diff --git a/app/views/projects/merge_requests/show/_versions.html.haml b/app/views/projects/merge_requests/show/_versions.html.haml index 49819519759..904452fcc4f 100644 --- a/app/views/projects/merge_requests/show/_versions.html.haml +++ b/app/views/projects/merge_requests/show/_versions.html.haml @@ -58,11 +58,11 @@ .monospace #{short_sha(merge_request_diff.head_commit_sha)} %small = time_ago_with_tooltip(merge_request_diff.created_at) - %li - = link_to merge_request_version_path(@project, @merge_request, @merge_request_diff), class: ('is-active' unless @start_sha) do - %strong - #{@merge_request.target_branch} (base) - .monospace #{short_sha(@merge_request_diff.base_commit_sha)} + %li + = link_to merge_request_version_path(@project, @merge_request, @merge_request_diff), class: ('is-active' unless @start_sha) do + %strong + #{@merge_request.target_branch} (base) + .monospace #{short_sha(@merge_request_diff.base_commit_sha)} - unless @merge_request_diff.latest? && !@start_sha .comments-disabled-notif.content-block |