Page MenuHomePhabricator

Embed search-ltr.png/search-rtl.png into CSS
Closed, ResolvedPublic

Description

They're a whole 214 bytes each!


Version: 1.17.x
Severity: enhancement

Details

Reference
bz34811

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 12:12 AM
bzimport set Reference to bz34811.
bzimport added a subscriber: Unknown Object (MLST).

Need to switch from <img> to probably a background-image directly on the <button>, which should be easy to CSS-embed.

  • Bug 34830 has been marked as a duplicate of this bug. ***

(Quote bug 34830 comment #0)

The styleversion is no longer maintained so this is useless.

(Found via https://developers.google.com/pagespeed/, complaining about query
strings on static resources; not entirely bad in general in MediaWiki but in
this case it really is useless as it's no longer maintained).

Perhaps this be done from CSS instead? (saves a bit of HTML, removes the query
string, and saves an http request when @embed'ed into CSS)