[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