stable version ?

DeepSky - January 15, 2009 - 16:28
Project:CSS Sprite Generator
Version:5.x-1.x-dev
Component:Miscellaneous
Category:feature request
Priority:critical
Assigned:Unassigned
Status:active
Description

I really want to use this module, but it does not seem to be working properly...the "coming soon" features, especially "Dynamically re-write CSS to implement the sprites." is what I need. From the module's code I can see is dealing with this, but probably has some minor bugs.
So, if not the lead developer, is anybody interested in releasing a stable version ?

#1

tjholowaychuk - January 16, 2009 - 16:57

I no longer work with PHP so I will not be finishing it unfortunately. This was primarily a prototype however anyone wanting to maintain it is certainly welcome, I will grant them CVS access.

#2

Flying Drupalist - January 22, 2009 - 23:05

subscribe

#3

tjholowaychuk - January 23, 2009 - 01:07

I should also note that this is not really even a practical thing to do. There are many reasons this flat out will not really work to any benefit, its possible to work yes, but its not tough at all just to create sprites in Photoshop, not to mention this way you KNOW they are compressed correctly, and optimized based on where they will be displayed on the site, as well as offsets via css to compensate for whitespace etc... its just not feasible

#4

Flying Drupalist - January 23, 2009 - 01:51

Yes, but it's a lot of work. What about something like this?

http://spritegen.website-performance.org/

#5

tjholowaychuk - January 23, 2009 - 04:49

That is a completely different concept. That simply builds a sprite collection for you, with out any knowledge of when / where they are going to be placed. Which would be completely useless as a module, you misewell just use that. Its not something I can easily explain over this, especially if you are not fluent with CSS or work with only basic layouts, but this is VERY unlikely to work properly/efficiently. Human intervention is the only proper way for CSS sprites to be implement.

As far as CSS Sprite creation goes, I would recommend people just take the 5 minutes to learn how to do it in photoshop, its really easy.

#6

Flying Drupalist - January 23, 2009 - 17:59

No I do work with sprites tjholowaychuk, but I find it a big hassle to have to edit it in photoshop every time I want to add something. The sprite gen DOES tell you where the sprites are placed, it generates the required css as well. I'm not sure I understand why this is not feasible?

#7

tjholowaychuk - January 24, 2009 - 06:50

Many reasons, but it is not something I can explain very well to someone who does not design.

#8

Flying Drupalist - January 24, 2009 - 16:39

I do design tjholowaychuk.

#9

tjholowaychuk - January 24, 2009 - 19:16

Well then you should understand the potential issues of attempting to automate the re-writing of a stylesheet. You would have to calculate properties inherited within any given block of css and adjust the box model accordingly, THEN you would have to allow sufficient padding within the sprites png to ensure that other sprites do not show up in this same background. If every sprite was simply going to placed on its own as the exact dimensions that it actually is as an image, then yeah sure thats super easy to do, but I can pretty much garantee this will not work with any reasonable amount of code.

#10

Flying Drupalist - January 24, 2009 - 19:24

Huh? You don't have to rewrite the stylesheet, you can create one with a class for each image. You leave enough space padding around each image to ensure that only one image shows up. I don't understand why you would have to readjust the box model. If it's just a background image the box model does not need to be touched.

#11

tjholowaychuk - January 24, 2009 - 19:46

haha blah... nvm, im not going to try and explain. If you were to use it like you mentioned you misewell just use the stand-alone ones found all over the web, theres no need for it to be a module if thats all it does.

 
 

Drupal is a registered trademark of Dries Buytaert.