[wp-trac] [WordPress Trac] #62879: get_custom_logo does not apply the aria-current attribute in all cases

WordPress Trac noreply at wordpress.org
Sun Feb 9 15:22:25 UTC 2025


#62879: get_custom_logo does not apply the aria-current attribute in all cases
---------------------------+----------------------------
 Reporter:  bschneidewind  |       Owner:  audrasjb
     Type:  defect (bug)   |      Status:  reviewing
 Priority:  normal         |   Milestone:  6.8
Component:  General        |     Version:
 Severity:  normal         |  Resolution:
 Keywords:  has-patch      |     Focuses:  accessibility
---------------------------+----------------------------

Comment (by siliconforks):

 Replying to [comment:9 sabernhardt]:
 > > Is that actually considered a valid, supported configuration (to have
 a posts page set but not a page set for the homepage)?  That doesn't
 really seem like it would be useful combination of settings to me.
 >
 > The edge case might be fine to allow, but I do not think the special
 posts page should have `aria-current` (or `current-menu-item` classes) on
 home links.
 >
 > When I assigned the page of posts without a static home page, the two
 URLs gave me duplicate content. With the Site Title block, I had the same
 link attributes at both URLs, too:
 > {{{
 > <p class="wp-block-site-title"><a href="http://localhost/wp671"
 target="_self" rel="home" aria-current="page">WP 6.7.1</a></p>
 > }}}

 Does this have anything to do with the edge case configuration, though?
 What happens if you ''do'' assign a static home page (in addition to a
 page of posts)?  Won't you have the same issue on the posts page?  (i.e.,
 the link says `aria-current="page"` when it shouldn't)

 This looks like it's a separate issue, with the Site Title block.

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


More information about the wp-trac mailing list