[wp-trac] [WordPress Trac] #40462: Add placeholders to wp_login_form()

WordPress Trac noreply at wordpress.org
Wed Sep 27 22:08:18 UTC 2017


#40462: Add placeholders to wp_login_form()
------------------------------------+--------------------------------------
 Reporter:  ramiy                   |       Owner:  voldemortensen
     Type:  enhancement             |      Status:  accepted
 Priority:  normal                  |   Milestone:  4.9
Component:  Login and Registration  |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch commit        |     Focuses:  accessibility, template
------------------------------------+--------------------------------------

Comment (by ramiy):

 Replying to [comment:20 afercia]:

 > @joedolson: As I see it, it will almost 100% be used to hide labels and
 use placeholders instead

 Not sure I agree with that direction of thinking. '''Core decisions should
 be based on more established arguments'''. I can easily contradict his
 claim - in most of my projects I use both labels and placeholders, like
 many of my colleagues.

 '''Besides, as long as placeholders are valid HTML attributes you cannot
 tell developers not to use them in their projects.'''

 From accessibility point of view '''we need to examine how WordPress Core
 uses placeholders, not how external developers might use them'''. I can
 tell you that WordPress core won't be affected from this patch because the
 default values are empty placeholders.

 So why add those attributes? because `wp_login_form()` , like many other
 [https://developer.wordpress.org/reference/ WP functions], is used not
 only by core but also by tens of thousands of external theme/plugins
 developers.

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


More information about the wp-trac mailing list