phpdevshell team mailing list archive

Thread
Date

[Bug 478840] Re: Invalid markup in cloud page template

To: phpdevshell@xxxxxxxxxxxxxxxxxxx
From: Greg <478840@xxxxxxxxxxxxxxxxxx>
Date: Mon, 27 Sep 2010 02:01:36 -0000
Reply-to: Bug 478840 <478840@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

Is this problem still valid with the new themes engine?

-- 
Invalid markup in cloud page template
https://bugs.launchpad.net/bugs/478840
You received this bug notification because you are a member of
PHPDevShell, which is subscribed to PHPDevShell.

Status in Open Source php gui type development framework.: New

Bug description:
The "cloud" theme page template wraps the module content in a table tag. The module may - or may not - supply table row elements for this table tag.

Most modules mix table rows and divs freely in their output, and this results in *very* invalid markup that just happens to be displayed correctly in a number of browsers today. The theme is supposed to be XHTML though, so we need to be pretty strict with the markup.

The table element can *only* contain table rows and, optonaly, thead and tbody elements to group the table rows. It is not allowed to contain elements such as div, p, hN etc.

I would suggest the table element is removed from the page template. Any module that supplies information to go into the table, needs to provide their own table element.