NOCC / Patches / #73 Use <th> for mail headers too

#73 Use <th> for mail headers too

Status: closed-fixed

Owner:

Labels: None

Priority: 5

Updated: 2006-09-26

Created: 2006-09-26

Creator: Tim Gerundt

Private: No

This patch adds <th> for the table header cells from
the mail headers too.
http://www.w3schools.com/tags/tag_th.asp

The only problem: Browsers use by default *bold* text
for header cells. Since you don't want this in your
case, we must add "font-weight: normal;" to the CSS
class "mailHeaderLabel". I know now, that you are no
friend from style changes, but I think in this case
wins the accessibility. And it should be difficult for
new NOCC releases (with new features), when you can't
change the themes a little bit. (I think on things like
"get rid of tables, switch to divs" from the TODO file.)

Greetings,
Tim Gerundt

Discussion

Tim Gerundt - 2006-09-26

original and altered files

use_th_for_mail_headers.zip

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Tim Gerundt - 2006-09-26

patch file

use_th_for_mail_headers.diff

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2006-09-26

assigned_to: nobody --> goddess_skuld

status: open --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2006-09-26

Logged In: YES
user_id=529507

Fixed in CVS.

It'll be included in next release.

But you can download daily snapshots at :
http://nocc.sourceforge.net/download/

Thanks for the patch.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Use <th> for mail headers too

Group

Searches

Help

#73 Use <th> for mail headers too

Discussion