Added aria-disabled=true for disabled pagination

mislav · May 9, 2021 · 31ed281 · 31ed281
1 parent 07a2647
commit 31ed281
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/will_paginate/view_helpers/link_renderer.rb b/lib/will_paginate/view_helpers/link_renderer.rb
@@ -71,7 +71,7 @@ def previous_or_next_page(page, text, classname)
         if page
           link(text, page, :class => classname)
         else
-          tag(:span, text, :class => classname + ' disabled')
+          tag(:span, text, :class => classname + ' disabled', :"aria-disabled" => true)
         end
       end
 

diff --git a/spec/view_helpers/action_view_spec.rb b/spec/view_helpers/action_view_spec.rb
@@ -127,7 +127,7 @@ def renderer.gap() '<span class="my-gap">~~</span>' end
   it "should match expected markup" do
     paginate
     expected = <<-HTML
-      <div class="pagination" role="navigation" aria-label="Pagination"><span class="previous_page disabled">&#8592; Previous</span>
+      <div class="pagination" role="navigation" aria-label="Pagination"><span class="previous_page disabled" aria-disabled="true">&#8592; Previous</span>
       <em class="current" aria-label="Page 1" aria-current="page">1</em>
       <a href="/foo/bar?page=2" aria-label="Page 2" rel="next">2</a>
       <a href="/foo/bar?page=3" aria-label="Page 3">3</a>
-Original file line number
+Diff line change
@@ Expand Up / @@ -71,7 +71,7 @@ def previous_or_next_page(page, text, classname) @@
             if page
               link(text, page, :class => classname)
             else
-              tag(:span, text, :class => classname + ' disabled')
+              tag(:span, text, :class => classname + ' disabled', :"aria-disabled" => true)
             end
           end
@@ Expand Down @@