When I looked at the xhtml code I discovered that the months are included by the “

”class odd. It will be a lot easier to design, when the months are include by

. And the title need to be included by

Can I change this by myself?

Comments

FPJ’s picture

FPJ commented

* Forget the c o d e t a g s *

When I looked at the xhtml code I discovered that the months are included by the “<tr>”class odd. It will be a lot easier to design, when the months are include by <th></th>. And the title need to be included by  <caption></caption> 
Can I change this by myself?
Susurrus’s picture

Susurrus commented

This is definitely changeable by a theme override function, which is one of the advantages of this code being inside a theme function. Some instructions on how to do this is here: http://drupal.org/node/11811. Also, you may wish to upgrade to 6.x-1.0 as it offers security fixes along with many other changes and I highly recommend updating. I think this function was actually changed to what you want in the 1.0 release.

Susurrus’s picture

Susurrus commented
Status: Active » Postponed

I'm not going to change this right now, at least, as the block looks significantly worse using the month/year as the caption and day titles as the header row. While it makes more sense for it to be arranged like this, I'm going to postpone it for now. You are, however, able to easily change any theme_* function using the override methods provided by your theming engine (normally PHPTemplate). If you modify the CSS for archive to theme the block nicely, then I would definitely commit this change, I just don't have time to mess with CSS styles right now.

Susurrus’s picture

Susurrus commented
Title: Theme Issue - Archive block » Use proper HTML semantic markup for table
Status: Postponed » Fixed

Fixed in 6.x and 5.x. Thanks for catching this.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.