diff --git a/app/frontend/scss/components/checks.scss b/app/frontend/scss/components/checks.scss index 6c24b24..26f3f5d 100644 --- a/app/frontend/scss/components/checks.scss +++ b/app/frontend/scss/components/checks.scss @@ -2,8 +2,4 @@ .action a { color: black; } - - .kind .octicon { - vertical-align: middle; - } } diff --git a/app/helpers/checks_helper.rb b/app/helpers/checks_helper.rb index 5976d8e..0eec22c 100644 --- a/app/helpers/checks_helper.rb +++ b/app/helpers/checks_helper.rb @@ -9,17 +9,24 @@ module ChecksHelper end def checks_sort_links(field) + %i[asc desc].map { |direction| + checks_sort_link(field, direction) + }.join + end + + def checks_sort_link(field, direction) + classes = "btn btn-light btn-sm mx-1 mx-1 px-1 py-0" current_sort_str = current_sort.to_a.join("_") - %i[asc desc].map { |direction| - sort = "#{field}_#{direction}" + sort = "#{field}_#{direction}" - icon = direction == :asc ? "chevron-up" : "chevron-down" - html = Octicons::Octicon.new(icon, class: "mx-1").to_svg.html_safe + icon = direction == :asc ? "chevron-up" : "chevron-down" + html = Octicons::Octicon.new(icon).to_svg.html_safe - filter_params = current_criterias.merge(sort: sort) - link_to_unless sort == current_sort_str, html, checks_path(filter_params) - }.join + filter_params = current_criterias.merge(sort: sort) + link_to_unless sort == current_sort_str, html, checks_path(filter_params), class: classes do + content_tag(:span, html, class: classes + " active") + end end def current_criterias diff --git a/app/views/checks/_table.html.erb b/app/views/checks/_table.html.erb index e164399..776d1de 100644 --- a/app/views/checks/_table.html.erb +++ b/app/views/checks/_table.html.erb @@ -4,18 +4,19 @@