Merge pull request #3316 from projectblacklight/2669-alt-totals

have totals in dashboard be for all items
projectblacklight · Dec 6, 2024 · 5273dff · 5273dff
2 parents 77852ae + 9d779d8
commit 5273dff
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/app/controllers/spotlight/accessibility_controller.rb b/app/controllers/spotlight/accessibility_controller.rb
@@ -8,11 +8,9 @@ class AccessibilityController < Spotlight::ApplicationController
     load_and_authorize_resource :exhibit, class: Spotlight::Exhibit
 
     def alt_text
-      @limit = 5
       # Sort by newest except for the homepage, which is always first
       pages_with_alt = @exhibit.pages.order(Arel.sql('id = 1 DESC, created_at DESC')).select { |elem| elem.content.any?(&:alt_text?) }
-      pages = params[:show_all] ? pages_with_alt : pages_with_alt.first(@limit)
-      @pages = pages.map { |page| get_alt_info(page) }
+      @pages = pages_with_alt.map { |page| get_alt_info(page) }
       @has_alt_text = @pages.sum { |page| page[:has_alt_text] }
       @total_alt_items = @pages.sum { |page| page[:can_have_alt_text] }
 

diff --git a/app/views/spotlight/accessibility/alt_text.html.erb b/app/views/spotlight/accessibility/alt_text.html.erb
@@ -55,10 +55,4 @@
       </tr>
     <% end %>
   </tbody>
-</table>
-
-<% unless params[:show_all] || @pages.length < @limit %>
-  <%= link_to '?show_all=true', class: 'ml-3' do %>
-    Show all<%= blacklight_icon('chevron_right') %>
-  <% end %>
-<% end %>
+</table>