[wp-trac] [WordPress Trac] #49059: Whitespace inside p element in wp-signup.php should be removed

WordPress Trac noreply at wordpress.org
Sat Dec 21 18:20:21 UTC 2019


#49059: Whitespace inside p element in wp-signup.php should be removed
--------------------------------+------------------------------
 Reporter:  henry.wright        |       Owner:  johnjamesjacoby
     Type:  enhancement         |      Status:  assigned
 Priority:  normal              |   Milestone:  5.4
Component:  Networks and Sites  |     Version:  3.0
 Severity:  normal              |  Resolution:
 Keywords:  close               |     Focuses:  multisite
--------------------------------+------------------------------
Changes (by azaozz):

 * keywords:  commit => close


Comment:

 > We should remove the whitespace because it isn't necessary.

 That's not entirely true :)

 White space in HTML is ignored. White space in the PHP code is used for
 formatting and better readability. Looking at the patches here, it seems
 they are not 100% compatible with the coding standards?

 Also, there are many many similar places in WP, do we convert them all to
 `echo '<string ...' . sprintf( .... ) . 'more strings';`? Don't think
 that's better readability, and if indeed it's needed, seems it will have
 to be added to the coding standards (the written standard, perhaps also to
 WPCS)?

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


More information about the wp-trac mailing list