[wp-trac] [WordPress Trac] #32510: Add HTML5 "required" attributes to the login form

[WordPress Trac]

#32510: Add HTML5 "required" attributes to the login form
----------------------------------------------+----------------------------
 Reporter:  aznadesign                        |       Owner:  joedolson
     Type:  enhancement                       |      Status:  reopened
 Priority:  normal                            |   Milestone:  6.3
Component:  Login and Registration            |     Version:  2.1
 Severity:  normal                            |  Resolution:
 Keywords:  has-patch required-fields commit  |     Focuses:  ui,
                                              |  accessibility
----------------------------------------------+----------------------------
Changes (by TimothyBlynJacobs):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 This change ended up breaking a plugin I maintain. We provide alternative
 ways to login that don't utilize the main username and password fields,
 but utilize the login page for ease of use. The other input fields are
 still in the DOM, but are no longer visible and do not contain a valid
 value. When the form is submitted, the browser fails validation on it.

 I'm reopening the ticket so we can evaluate if that is a backward
 incompatibility we want to avoid. Or if we are ok with that breakage. My
 guess is this could impact other plugins that additional login methods.

 I think you could argue not removing the other input fields is
 #doingitwrong. We can add and remove the core input fields from the dom
 instead.

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