Color module seems to have been broken after http://drupal.org/node/137211. Currently it cannot correctly create a stylesheet for a new Garland color scheme. To reproduce:

1. Go to admin/build/themes/settings/garland and create a new scheme
2. Submit.
3. Check your new files/color/garland- directory. All the files are generated correctly but the style.css is a blank file.

The attached patch keeps the ability to name the stylesheet whatever you want (not just style.css), and instead takes the *first* stylesheet from the theme as the source for color modifications, since now we can have multiple stylesheets defined per theme.

CommentFileSizeAuthor
color_stylesheet.patch705 bytesquicksketch

Comments

dries’s picture

dries commented

How do we know that the first stylesheet is the proper one?

quicksketch’s picture

quicksketch
he/him
commented

I'm not familiar with the inner workings of color module. I'm not sure if it would be safe to operate on all the css files in a theme or not. Should there be another value in the info file (or a hook) for which css file should be used?

hass’s picture

hass commented

Please look into http://drupal.org/node/113607 and http://drupal.org/node/175121. This patches should fix this issue, too.

I'd would mark this issue as won't fix and concentrate on the other two, while they solve tons of other outstanding bugs together.

hass’s picture

hass commented
Status: Needs review » Closed (won't fix)

go to http://drupal.org/node/113607

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented

We had the same issue. Here are the steps we did to fix this:
1. Go to /admin/build/themes/settings/garland
2. Click on “Save” button. This will overwrite the database settings for the color module. Drupal will output the following message:

* The configuration options have been saved.
* The directory ***/files/color has been created.
* The directory ***/files/color/garland-*** has been created.

3. The next page will automatically reload. And Garland is back to normal.