Patches item #1415956, was opened at 2006-01-26 22:23
Message generated for change (Comment added) made by akuchling
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1415956&group_id=103
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Web UI
Group: Mailman 2.1
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Bryan Carbonnell (carbonnb)
Assigned to: Nobody/Anonymous (nobody)
Summary: Sitewide headers/footers & XHTML Compliant Web UI
Initial Comment:
This patch was borne out of a request I received to
make the Mailman UI fit the look of the web site. This
patch allows you to set a site wide header and footer.
This allows you to pretty much make the MM UI look like
any other site. While I was at it I also made the web
UI XHTML compliant.
Once you patch your source and install it, all you need
to do is edit the html files in the templates/en
directory. Most of the pages will get the header and
footers from the site-header.html and site-footer.html
files, but some of the HTML files already contain theor
own header/footer so you will need ot edit some of
these files as well.
Since this also adds XHTML compliance, this superceeds
patch #116035
----------------------------------------------------------------------
Comment By: A.M. Kuchling (akuchling)
Date: 2007-05-05 21:08
Message:
Logged In: YES
user_id=11375
Originator: NO
A branch for CSS work has been created, modernize_21_webui.
I've committed most of this patch.
The patch to the following files didn't apply: templates/en/roster.html
templates/en/subscribe.html templates/en/listinfo.html
templates/en/admindbdetails.html .
There have been changes to the files since the patch was made, so they'll
basically have to be re-done from scratch. Bryan, do you remember all the
various changes you made, or should I try to disentangle the original and
patched versions?
----------------------------------------------------------------------
Comment By: A.M. Kuchling (akuchling)
Date: 2007-05-03 20:52
Message:
Logged In: YES
user_id=11375
Originator: NO
The patch looks pretty good, though I'll suggest some refinements and some
decisions need to be made.
Net effects of the patch:
Adds XHTML compliance.
Adds site-header.html/site-footer.html to the templates/en directory
and usage of CSS instead of <font> tags.
Still uses tables for page layout.
Adds CSS_FILE to mm_cfg.py.
Requires addition of a /css/ directory to the web tree.
Possible issues:
Many I18N strings change, but only by folding HTML markup to lowercase.
A bunch of search-and-replacing should be able to update all the
language-specific templates.
Need to decide upon class naming scheme; the scheme used isn't
consistent (mm_web_adminitem_color in one case, mmErrorBackground
in another).
Could retarget it to use the <div id="mailman"> suggestion
and '#mailman error' selectors.
Possible bug: AddSpanStyle, AddDivStyle: won't generate <div>,<span>
if no 'css_class' is provided. Presumably all current callers get this
right; should the class work properly if there's an error, though?
Compatibility issues:
The patch removes the colour settings (WEB_HEADER_COLOR,
WEB_ERROR_COLOR, WEB_ADMINITEM_COLOR, etc.) -- how to handle backward
compatibility? That would require substituting into the .css file, or
adding a 'style' attribute.
WEB_HEADER_COLOR -- mmHeader
WEB_ERROR_COLOR -- replaced by mmErrorBackground CSS class
WEB_ADMINITEM_COLOR -- replaced by mmRight/mmLeft/mm_web_adminitem_color/
mmAdminItem
Features not present that I thought of:
* allow adding an embedded stylesheet
* allow adding JavaScript links or text to the <head>
* adding an onLoad to the body (perhaps not necessary -- can it be done in
a <script> tag using DOM Events?)
----------------------------------------------------------------------
Comment By: Bryan Carbonnell (carbonnb)
Date: 2006-01-28 23:07
Message:
Logged In: YES
user_id=449953
Updated both patches to remove some of my site specific HTML
coding.
----------------------------------------------------------------------
Comment By: Bryan Carbonnell (carbonnb)
Date: 2006-01-26 22:31
Message:
Logged In: YES
user_id=449953
I am also uploading a second version of this patch. This
second version will add the sitewide headers/footer to
source that has been patched with the ht://dig integration
patches.
At the time I posted this, the official ht://dig integration
patches were not released, but the MM2.1.6 patches appear to
work with MM 2.1.7, so they are what were used.
to apply this patch use this command from your mailman
source directory:
patch -p1