[wp-trac] [WordPress Trac] #63581: paginate_links: The current page should be an a element, not a span element

[WordPress Trac]

#63581: paginate_links: The current page should be an a element, not a span element
--------------------------+------------------------------
 Reporter:  wildworks     |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  General       |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  2nd-opinion   |     Focuses:  accessibility
--------------------------+------------------------------
Changes (by sabernhardt):

 * keywords:   => 2nd-opinion


Comment:

 The W3C Pagination and Breadcrumbs pages follow **one of**
 [https://scottaohara.github.io/a11y_breadcrumbs/ Scott O'Hara's breadcrumb
 pattern examples]. Scott's third example assigns `aria-current` to the
 `li` element, without a link, and the usage notes present arguments both
 for and against linking.

 Changing the function's default behavior now would require a clear and
 significant improvement. I do not think the linked pattern is //much//
 better, but others might disagree.

 However, //choosing// to link the current page probably could be easier. I
 only know one way to alter the markup: using `preg_replace()` within the
 `paginate_links_output` filter.

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