Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#12876 closed enhancement (fixed)

Remove unused images / compress other images

Reported by: ocean90's profile ocean90 Owned by:
Milestone: 3.0 Priority: low
Severity: normal Version:
Component: Optimization Keywords:
Focuses: Cc:

Description

In wp-admin/images/ exists some images which we didn't need (anymore).
If we clean up the code, we should clean up the images too.

Unused images

  • wp-admin/images/
    • ico-add.png
    • ico-close.png
    • ico-edit.png
    • ico-viewpage.png
    • loading.gif (think we have wpspin_*.gif for it)
    • loading-publish.gif (think we have wpspin_*.gif for it)

I think there are more unused images, so use this ticket to clean up the images.

Compress images

Some images are already compressed, but not all. I use PunnyPNG for compressing and didn't have any problems with it. Some examples:

  • wp-admin/images/fav.png | orig: 214 bytes after: 96 bytes
  • wp-admin/images/imgedit-icons.png | orig: 9,607 bytes after: 7,105 bytes

Totaly we can save 16.3 KB which are 11% (only wp-admin/images/).

Maybe a committer can do this or I can upload a zip with all compressed images.

Change History (7)

#1 @nacin
15 years ago

The issue with removing loading.gif and loading-publish.gif is that a plugin may be using them. (Indeed, if a plugin developer goes looking for an ajax loading animated gif in wp-admin/images, they're probably more likely to notice loading*.gif than they are wpspin_*.gif.)

No real harm in keeping those, as they add very little weight to the package, and add nothing to page load -- unless a plugin is using them, of course.

On the other hand, the other ones are from Woo, [13267]. Those can go.

Compressing -- definitely something we should do. I don't really like PunyPNG because it's a site, versus a script like pngcrush.

#2 @t31os_
15 years ago

Please leave the loading gif, i've always preferred it over the new wpspin images (which impo look awful).

#3 @ocean90
15 years ago

Ok, loading.gif was a bad idea, but a new one browse-happy.gif, see [14233]

#4 @automattor
15 years ago

(In [14396]) Remove browse-happy.gif. see #12876. props ocean90

#5 @nacin
15 years ago

(In [14397]) Remove browse-happy.gif. see #12876. props ocean90

#6 @nacin
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [14453]) Remove unused woo nav menu icons. fixes #12876

#7 @nacin
15 years ago

Let's talk compression and other improvements in 3.1.

Note: See TracTickets for help on using tickets.