diff options
author | Robert Speicher <robert@gitlab.com> | 2017-05-04 17:07:59 +0300 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2017-05-04 17:07:59 +0300 |
commit | 50b89fc5b0dbd165d8e734c48639287428cad576 (patch) | |
tree | dfbb81a839d1311d4d7cd9c809b0c8b8812be517 /app | |
parent | 37b1b14a167de18e35810f4728bbf8d70920ffca (diff) | |
parent | fdacc4ee6a3341b2c44ddd85a41f2a04d0d417ad (diff) |
Merge branch 'tags-sort-default' into 'master'
Fixed tags sort dropdown being empty
Closes #31618
See merge request !11055
Diffstat (limited to 'app')
-rw-r--r-- | app/helpers/sorting_helper.rb | 8 | ||||
-rw-r--r-- | app/views/projects/tags/index.html.haml | 17 |
2 files changed, 16 insertions, 9 deletions
diff --git a/app/helpers/sorting_helper.rb b/app/helpers/sorting_helper.rb index 2fda98cae90..4882d9b71d2 100644 --- a/app/helpers/sorting_helper.rb +++ b/app/helpers/sorting_helper.rb @@ -70,6 +70,14 @@ module SortingHelper } end + def tags_sort_options_hash + { + sort_value_name => sort_title_name, + sort_value_recently_updated => sort_title_recently_updated, + sort_value_oldest_updated => sort_title_oldest_updated + } + end + def sort_title_priority 'Priority' end diff --git a/app/views/projects/tags/index.html.haml b/app/views/projects/tags/index.html.haml index 7f9a44e565f..56656ea3d86 100644 --- a/app/views/projects/tags/index.html.haml +++ b/app/views/projects/tags/index.html.haml @@ -1,4 +1,5 @@ - @no_container = true +- @sort ||= sort_value_recently_updated - page_title "Tags" = render "projects/commits/head" @@ -14,16 +15,14 @@ .dropdown %button.dropdown-toggle{ type: 'button', data: { toggle: 'dropdown'} } %span.light - = projects_sort_options_hash[@sort] + = tags_sort_options_hash[@sort] = icon('chevron-down') - %ul.dropdown-menu.dropdown-menu-align-right - %li - = link_to filter_tags_path(sort: sort_value_name) do - = sort_title_name - = link_to filter_tags_path(sort: sort_value_recently_updated) do - = sort_title_recently_updated - = link_to filter_tags_path(sort: sort_value_oldest_updated) do - = sort_title_oldest_updated + %ul.dropdown-menu.dropdown-menu-align-right.dropdown-menu-selectable + %li.dropdown-header + Sort by + - tags_sort_options_hash.each do |value, title| + %li + = link_to title, filter_tags_path(sort: value), class: ("is-active" if @sort == value) - if can?(current_user, :push_code, @project) = link_to new_namespace_project_tag_path(@project.namespace, @project), class: 'btn btn-create new-tag-btn' do New tag |