Hi Patrik. Thanks for your answer.
> * presentational IDs such as <div id="left"> are a bad choice. IDs and > classes should be chosen based on the function of that piece of markup, > not its appearance; > yes, I know. They have been called that way in the template. Never got around renaming them. > * very weird navigation with the dl...you have some empty (spacer?) > elements there like <dt> - what's that then? if you want space, > shouldn't you use CSS? same goes for the arrows on some of the other > <dt>s; generally, the structure of that navigation is odd; > The idea is, to have the main menu items in dt, and the submenu items in dd. Main menu items can be flipped open and close, so the submenu items can be hidden. Klicking on the main menu items shows or hides the submenu items. The menu is created dynamically out of configuration files, where we have to allow empty entries for the mainmenu items. Using dl for this is not OK? > * the caption on that calendar is not a caption at all; it's a sub > navigation; don't mark it up as a table caption, make it a separate > element (an ordered or, at a stretch, unordered list seems most natural > to me) > I'll change that. > * the table could do with clearly defined <thead> and <tbody> sections > Done. > En schoene, > Ebenso. Cheers, Michael Vogt ****************************************************** The discussion list for http://webstandardsgroup.org/ See http://webstandardsgroup.org/mail/guidelines.cfm for some hints on posting to the list & getting help ******************************************************
