Calendar module creates two html tables (one for the head with the month and another for the body with the days).

It's a bad data representation: it's not xhtml valid (there's an empty tbody tag), it's not wcag valid (and it's not accessible for blind people).
There should be only a table, with a head (the month) and a body (the days).

By the way, thanks for all ;)

Comments

Ufic’s picture

Assigned: Ufic » Unassigned
m1mic’s picture

@Ufic

The empty tbody tag wasn't coming from the Calendar module, it was a bug in Drupal core itself. This was fixed in Drupal 6, and I backported the patch to 5 (Node #171461). Earlier today (June 23, 2008), that patch was applied to Drupal 5. The next time an official release comes out, the tbody problem in 5 will go away.

m1mic’s picture

Title: Produced code is not xhtml valid » Accessibility: Two HTML tables
Category: bug » feature

Moving this to a feature request. The XHTML bug wasn't generated because of the Calendar module, but was because of the aforementioned tbody issue in Drupal 5 core.

KarenS’s picture

Status: Active » Fixed

Sounds like it's fixed, then (there's nothing else to do here).

The handling for this has been altered in later versions to try to avoid the problem by adding the navigation to the calendar table (in the D5.2 version) or by getting rid of the table altogether for the navigation and doing it using divs (in the D6 version).

Anonymous’s picture

Status: Fixed » Closed (fixed)

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