Merge pull request #259 from MITLibraries/use-147-move-tabs

Move tab filters to search form

Author: jazairi

app/assets/stylesheets/partials/_search.scss

@@ -192,3 +192,7 @@
   margin: 2rem 0;
   text-align: center;
 }
+
+.tab-link {
+  color: #fff;
+}

app/views/search/_form.html.erb

@@ -27,4 +27,6 @@
     </aside>
 <% end %>
 
+<%= render partial: 'search/source_tabs' if @results.present? %>
+
 <%= javascript_include_tag "search_form" %>

app/views/search/_source_tabs.html.erb

@@ -0,0 +1,10 @@
+  <!-- Tab Navigation -->
+  <div id="tabs" class="tab-navigation top-space">
+    <%= link_to "Primo", results_path(params.permit(:q, :per_page, :page).merge(tab: 'primo')), 
+                class: "tab-link #{'active' if @active_tab == 'primo'}", 
+                data: { turbo_frame: "search-results", turbo_action: "advance" } %>
+    <%= link_to "TIMDEX", results_path(params.permit(:q, :per_page, :page).merge(tab: 'timdex')), 
+                class: "tab-link #{'active' if @active_tab == 'timdex'}", 
+                data: { turbo_frame: "search-results", turbo_action: "advance" } %>
+  </div>
+  

app/views/search/results.html.erb

@@ -12,16 +12,6 @@
 
   <%= render(partial: 'shared/error', collection: @errors) %>
 
-  <!-- Tab Navigation -->
-  <div id="tabs" class="tab-navigation top-space">
-    <%= link_to "Primo", results_path(params.permit(:q, :per_page, :page).merge(tab: 'primo')), 
-                class: "tab-link #{'active' if @active_tab == 'primo'}", 
-                data: { turbo_frame: "search-results", turbo_action: "advance" } %>
-    <%= link_to "TIMDEX", results_path(params.permit(:q, :per_page, :page).merge(tab: 'timdex')), 
-                class: "tab-link #{'active' if @active_tab == 'timdex'}", 
-                data: { turbo_frame: "search-results", turbo_action: "advance" } %>
-  </div>
-
   <%= render(partial: 'trigger_tacos') if tacos_enabled? %>
 
   <%= turbo_frame_tag "search-results" do %>