[wp-trac] [WordPress Trac] #30274: wp-login.php Logo Image Use "img" tag rather than CSS background

WordPress Trac noreply at wordpress.org
Sun Feb 5 07:26:38 UTC 2017


#30274: wp-login.php Logo Image Use "img" tag rather than CSS background
------------------------------------+-----------------------
 Reporter:  themightymo             |       Owner:
     Type:  enhancement             |      Status:  reopened
 Priority:  normal                  |   Milestone:
Component:  Login and Registration  |     Version:  4.0
 Severity:  normal                  |  Resolution:
 Keywords:                          |     Focuses:  ui
------------------------------------+-----------------------
Changes (by pingram3541):

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


Comment:

 There is an even more important reason to reconsider this.  Static payload
 of images in css.

 Currently the only way to use a custom logo is to override it with css or
 use an alternate form of login altogether.  Core loads a combined 4.6kb
 payload for both logo images in wp-admin.css and it isn't practical to
 deregister this file, provide our own to eliminate the extra payload, and
 then re-enqueue the modified version of the file because later it can
 become outdated with future releases of WP.

 If an img element was used we can filter the url using functions.php or a
 plugin or possibly even enhance the ability to modify the argument array
 of wp_login_form( $args ) passing the image url.

 Plugin authors know the risk of an ever changing core so I think the
 excuse is not really that strong of an argument when we could be doing it
 in a more efficient, modern way.

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


More information about the wp-trac mailing list