String freeze: color.module - further improvements to Drupal 6 module help documentation

These changes look good to me. I think the Garland and Minelli stuff look better as you have it (as a parenthetical), but, as you say, someone may have another view of that. Thank you for reversing the not on that sentence, that's much more readable.

In
+ $output .= '<p>'. t('To change the color settings for a compatible theme, select the "configure" link for the theme on the main <a href="@url">themes administration page</a>.', array('@url' => url('admin/build/themes'))) .'</p>';

you say "main themes administration page" to perhaps distinguish from the individual theme configuration pages? I'm not sure "main" adds much there, since none of the individual theme configuration pages is the "themes administration page" -- I mean, you are already linking to a specific page, here. That's a very minor point though and doesn't even deserve CNW.

$ patch -p0 < color.module.help_.patch
(Stripping trailing CRs from patch.)
can't find file to patch at input line 3
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|--- modules\color\color.module.1.32 Sun Dec 16 11:54:09 2007
|+++ modules\color\color.module Sun Dec 16 12:43:23 2007
--------------------------
File to patch: modules/color/color.module
patching file modules/color/color.module
Hunk #1 FAILED at 7.
1 out of 1 hunk FAILED -- saving rejects to file modules/color/color.module.rej

While we are at this, let's remove the "main" word :)

Rerolled the patch, and removed "main."

Great. I've committed this to CVS HEAD.