2 messages in net.sourceforge.lists.courier-sqwebmailRe: [sqwebmail] XHTML validation: fol...
FromSent OnAttachments
Pawel TeczaJul 5, 2006 4:49 am 
Sam VarshavchikJul 5, 2006 2:02 pm 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:Re: [sqwebmail] XHTML validation: folder_err_msgActions...
From:Sam Varshavchik (mrs@courier-mta.com)
Date:Jul 5, 2006 2:02:56 pm
List:net.sourceforge.lists.courier-sqwebmail

Pawel Tecza writes:

Hi Sam,

This is my first XHTML validation related patch. I have found that in function folder_list2() (folder.c file) which displays folder error message you don't close <p> tag.

I fixed it and moved HTML code from this function to folders.html template, because IMHO hardcoded HTML code is not a good idea. I have also unified HTML code for error messages in the template. The "size" and "color" attributes were removed, because there are unnecessary if you use "class" attribute.

Any comments? :)

Nope.

This are the kinds of things I prefer to see. Small changes, that can be proven to be correct just by eyeball alone.