[Fixed] XHTML validation

[polyvios_s]

Has anyone tried to make the printed page validate with XHTML?

Would it be breaking user agreement if I tried to make mine work like that?

[Jordan]

Not at all. You can do whatever you like ionFiles. If you do make it XHTML valid, let us know what you changed and/or post the final version here so that we can add your changes to the official version.

[polyvios_s]

Didn't figure I could make this work but the downloads page is pretty much configured with the exception of tr id being defined a couple of times more than it should.

Just erase $rowId from lines 358 and 471 of ionfiles.php

Before - Line 358:

print "<tr id=\"$rowId\"><td width=\"100%\" colspan=\"$colCount\" class=\"sectiontableheader\">"; 

Before - Line 471:

print "<tr id=\"$rowId\"><td colspan=\"$colCount\"><br /></td></tr>"; 

After - Line 358:

print "<tr><td width=\"100%\" colspan=\"$colCount\" class=\"sectiontableheader\">"; 

After - Line 471:

print "<tr><td colspan=\"$colCount\"><br /></td></tr>"; 

Not sure if this is a proper fix or a workaround, not even sure if it will work for different setups than mine... but for me it validates through w3c now if I can just figure out why I cant style "contentheading"

[Jordan]

Excellent, I'll make these changes to the official version for the next release. You should be able to change the style of "contentheading" in your CSS files.

[polyvios_s]

the "contentheading" thing was a conflict I probably wouldn't have noticed without ionfiles, had it defined twice

Found a new cause of XHTML verification failure though, when not using a single word for a category title one of the table id's fails since spaces are not allowed. Two ways around this that I can think of:

1. Couple of codelines to strip the spaces before using the title for the table id.
2. using category id instead of the title for the table id

I'll take a look at it after the weekend when I'll be working on my downloads area, for now my workaround is simply to use single word category titles!