Patch attached that suppresses the notice.

CommentFileSizeAuthor
color_list_explode_notice.patch766 byteshass

Comments

hass’s picture

How to get this notice:

1. Go to theme settings in Garland
2. Use a different color schema then standard
3. Press "save"
4. Notice is displayed on top.

pasqualle’s picture

I can't see the notice. Need more info on how to reproduce the bug..

hass’s picture

Do you have PHP configured with E_ALL only? Simply follow the above steps. All you need is a CSS file that does not contain a Color Module: Don't touch section. You won't get the notice in garland, while this have this section.

pasqualle’s picture

Status: Needs review » Reviewed & tested by the community

Is 6.x-dev is configured with E_ALL, or I need to change something?
In steps you wrote to select garland theme, and in your last post you wrote select something else than garland.

---
OK, figured out. Inserted an additional space into "Color Module: Don't touch" string in style.css and I can see the notice.

patch works as advertised
(hope, that windows-style line endings is not be a problem)

dries’s picture

Status: Reviewed & tested by the community » Needs work

Fixing something with @ is not the right thing to do.

hass’s picture

Yes i know... :-(. i fighted over 1 week with me and Google to make this better, but i haven't found a better solution. If you know a solution, please share your idea... i'm not an E_ALL guru, sorry.

pasqualle’s picture

The hard-coded "Color Module: Don't touch" string should be completely removed. But I am not familiar with the functionality.

hass’s picture

This hard coded string is a functionality we cannot change or it will break the Garland theme and others that like to depend on color module...

hass’s picture

Status: Needs work » Closed (duplicate)