Bartik's color integration misses 'base' key

Seems like this has improved, but is still a problem. I just tried the same and got:

Notice: Undefined index: base in _color_rewrite_stylesheet() (line 437 of modules/color/color.module).
Notice: Undefined index: base in _color_rewrite_stylesheet() (line 437 of modules/color/color.module).

Not sure if this is a problem w/Bartik or the color module.

Smerrill???

This is a problem with color module and afair it was present before any patches were introduced. It has quite serious implications since colors not defined in color.inc are shifted relatively to base color.

You can work around this issue by declaring base color in color.inc to be slightly different than base color declared in CSS files, but in many cases this will cause colors to be shifted incorrectly.

I ran into this problem too when playing with Bartik.

It is mentioned here:
#776684: The color.module preview HTML is hardcoded

but I've created an issue for it here:
#789554: Notice: Undefined index: base in _color_rewrite_stylesheet() in line 437

So..... is there anything we need to do for Bartik to help with this? Or do we close this issue as won't fix, because it's a core color module thing, and not a Bartik thing.

Well, I'm guessing that $base isn't defined where it should be in Bartik because it doesn't show up as an error when switching colors in Garland.

So I don't think it can be closed, but I don't really know enough about D7 theming to say for sure.

The error does not appear in Garland because it is not using base color (#0072b9) in any of its stylesheets. Try adding #blah { background-color: #0072b9; } to Garland's style.css and you will get the same error:

Notice: Undefined index: base in _color_rewrite_stylesheet() (line 416 of /opt/lampp/htdocs/demosite-1/modules/color/color.module).

Guess it can be closed then here in Bartik.

See #1236098-43: Notice: Undefined index: 'base' in _color_rewrite_stylesheet()