[wp-trac] [WordPress Trac] #41859: Add aria-current to the paginated posts links output by wp_link_pages()

WordPress Trac noreply at wordpress.org
Tue Nov 28 20:03:37 UTC 2017


#41859: Add aria-current to the paginated posts links output by wp_link_pages()
-------------------------------------------------+-------------------------
 Reporter:  afercia                              |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  5.0
Component:  Themes                               |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  aria-current needs-patch dev-        |     Focuses:
  feedback                                       |  accessibility
-------------------------------------------------+-------------------------
Changes (by antonioeatgoat):

 * keywords:  aria-current needs-patch => aria-current needs-patch dev-
     feedback


Comment:

 The patch uses by default some of the parameters used by Twenty Seventeen,
 however this upsets the theme, because there is a repeated nested span
 with the same class. So likely not the best way to achieve it, maybe
 changing the class name used in the would be a solution.

 Also I am not a big fun of the solution {{{ str_replace( '<a href="', '<a
 class="page-number" href="', $link ); }}}, however is the only way that I
 see for now with the current structure of the functions, but it would be
 very nice to improve consistency between {{{paginate_links()}}} and
 {{{wp_link_pages()}}} because they use parameters and class
 positions/names in a quite different way, despite they do basically the
 same thing.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/41859#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list