[wp-trac] [WordPress Trac] #51786: Accessibility issue with the logo on the login page

[WordPress Trac]

#51786: Accessibility issue with the logo on the login page
----------------------------------------+----------------------------
 Reporter:  roytanck                    |       Owner:  joedolson
     Type:  enhancement                 |      Status:  accepted
 Priority:  normal                      |   Milestone:  6.7
Component:  Login and Registration      |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch has-testing-info  |     Focuses:  accessibility
----------------------------------------+----------------------------

Comment (by sabernhardt):

 I think I would rather remove the `header` and `main` elements from this
 patch. Landmarks are a [https://dequeuniversity.com/rules/axe/3.5/region
 best practice], but the login pages on **most** sites essentially contain
 only one region right now. I'm more concerned about the possibility of
 adding semantic elements when authors have already created them (for
 example, //within// the output of `login_header` or `login_footer` hooks).
 At worst, leaving them out of the patch is a lack of improvement to
 navigating the page at this time.

 Then the code could omit the hidden heading for the email confirmation
 screen without leaving an empty `header`.
 {{{
 <?php
   if ( 'confirm_admin_email' !== $action ) :
 ?>
     <h1 class="screen-reader-text"><?php echo $title; ?></h1>
 <?php
   endif;
 ?>
 }}}

 If the login page ever adds the admin toolbar (#48831), that would make
 regions and/or a skip link more helpful/necessary.

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