[Mailman-Developers] Styling patch

[Mads Kiilerich]

Terri Oda wrote, On 05/05/2007 05:24 PM:
>> It presents one big compatibility issue that will probably need to be
>> fixed.  The patch removes the colour settings (WEB_HEADER_COLOR,
>> WEB_ERROR_COLOR, WEB_ADMINITEM_COLOR, etc.)  and replaces them with
>> CSS class declarations.
>>
>> It's probably unacceptable to break the use of these settings, so
>> there needs to be some compatibility fix.  Colours could be
>> substituted into the .css file somehow, or a 'style' attribute could
>> be added as well as a 'class' attribute.  Does anyone have suggestions
>> for what to do?
>>     
>
> Some thoughts with positive (+) and negative (-) notes:
>
> Have the CSS generated by a CGI (which could then use these variables  
> if they're available).
> - extra processing for each web request
> + could lay groundwork to make it possible to change settings on a  
> per-list (or per virt-domain) basis rather than site-wide.
>   

Perhaps that part of the CSS could be included inline in the already 
cgi-generated html header. It will probably only be a couple of lines. 
If placed before inclusion of custom css then customizers can override 
it if they want to. (That would be very close to how it is done in Plone.)

IMHO that addresses the only minus of this solution.

Just a thought...

/Mads