Merge pull request #806 from sul-dlss/fix-search-button

Use local component for search button

app/components/arclight/search_bar_component.html.erb

@@ -23,6 +23,6 @@
   <% end if append? %>
 
   <% c.with_search_button do %>
-    <%= search_button %>
-  <% end if respond_to?(:search_button?) && search_button? # only BL8 has this option %>
+    <%= render SearchButtonComponent.new(id: "#{@prefix}search", text: scoped_t('submit')) %>
+  <% end %>
 <% end %>

app/components/landing_page_search_bar_component.rb

@@ -8,4 +8,8 @@ def initialize(**kwargs)
     # Default to grouping search results by collection.
     @params = @params.merge(@params, group: true)
   end
+
+  def search_button
+    render SearchButtonComponent.new(id: "#{@prefix}search", text: scoped_t('submit'))
+  end
 end

app/components/search_button_component.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+# Subclasses Blacklight's SearchButtonComponent so we can set the
+# button text to visually-hidden at all screen sizes
+class SearchButtonComponent < Blacklight::SearchButtonComponent
+  def call
+    tag.button(class: 'btn btn-primary search-btn', type: 'submit', id: @id) do
+      tag.span(@text, class: 'visually-hidden submit-search-text') +
+        render(Blacklight::Icons::SearchComponent.new)
+    end
+  end
+end