In the plain widget, there are 4 images. Performance practice says that we should use sprite, and this case is particularly right. All of them are used as CSS background image for fixed-size span, thus not so hard for CSS sprite ;)

CommentFileSizeAuthor
#7 0001-feature-request-785344-by-jcisio-Added-CSS-sprites-f.patch8.24 KBmarvil07
#4 widgets.zip4.26 KBdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marvil07’s picture

marvil07 CreditAttribution: marvil07 commented
Title: CSS Sprite » CSS sprites for alternate and plain widgets

good idea, this applies also to alternate widget :-)

marvil07’s picture

marvil07 CreditAttribution: marvil07 commented
Priority: Normal » Minor
marvil07’s picture

marvil07 CreditAttribution: marvil07 commented
Version: 6.x-2.x-dev » 6.x-3.x-dev
Priority: Minor » Normal
droplet’s picture

droplet CreditAttribution: droplet commented
FileSize
widgets.zip4.26 KB

here's the patched files.
i don't know how to add/remove files with CVS, so simply zip them all.

droplet’s picture

droplet CreditAttribution: droplet commented
Status: Active » Needs review
marvil07’s picture

marvil07 CreditAttribution: marvil07 commented
Status: Needs review » Fixed
Issue tags: +frontend performance

Great!

I have just committed your changes(see this patch for details) to 3.x and backported it to 2.x.

Thanks for the sprites.

marvil07’s picture

marvil07 CreditAttribution: marvil07 commented
FileSize
0001-feature-request-785344-by-jcisio-Added-CSS-sprites-f.patch8.24 KB

Upps, forgot the patch

Status: Fixed » Closed (fixed)
Issue tags: -frontend performance

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +frontend performance

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.