Merge pull request #3203 from projectblacklight/explicit-classes

Explicitly pass classes on the search widgets

Author: jcoyne

app/assets/stylesheets/blacklight/_controls.scss

@@ -1,9 +1,5 @@
 .search-widgets {
   display: flex;
-
-  > * {
-    @extend .mx-1;
-  }
 }
 
 .sort-pagination,

app/components/blacklight/response/sort_component.rb

@@ -22,6 +22,7 @@ def dropdown
                  param: @param,
                  choices: @choices,
                  id: @id,
+                 classes: @classes,
                  search_state: @search_state,
                  selected: @selected
                ))

app/components/blacklight/search/per_page_component.rb

@@ -23,6 +23,7 @@ def dropdown
                  param: :per_page,
                  choices: per_page_options_for_select,
                  id: 'per_page-dropdown',
+                 classes: ['mx-1'],
                  search_state: @search_state,
                  selected: current_per_page,
                  interpolation: :count

app/views/catalog/_sort_widget.html.erb

@@ -1,6 +1,7 @@
 <% if show_sort_and_per_page? %>
   <%= render(Blacklight::Response::SortComponent.new(
     choices: active_sort_fields.map { |key, config| [sort_field_label(config.key), key] },
+    classes: ['mx-1'],
     search_state: search_state,
     selected: current_sort_field&.key)) %>
 <% end %>