forum.coppermine-gallery.net

Dev Board => cpg1.4 Testing/Bugs => cpg1.4 Testing/Bugs: FIXED/CLOSED => Topic started by: kaptainkory on August 02, 2005, 03:26:30 am

Title: px text in <td> width attribute invalid
Post by: kaptainkory on August 02, 2005, 03:26:30 am

It is invalid to use px in the width value for <td> tags:

BAD:

Code: [Select]

<td width="48px">
GOOD:

Code: [Select]

<td width="48">

Also, for <table> tags:

BAD:

Code: [Select]

<table cellspacing="48px" cellpadding="48px">
GOOD:

Code: [Select]

<table cellspacing="48" cellpadding="48">
In themes.inc.php, if you do a search, you'll find plenty of these that need changed.

Thanks.

Title: Re: px text in <td> width attribute invalid
Post by: donnoman on August 02, 2005, 07:20:59 am

I think we discussed this at least once on the dev list, bottom line there was confusion between css, and based on the fact that the px's didn't trip the XHTML validator I left it alone.

I believe as you do that when used in the context of xhtml attributes there should be no size qualifier, only used when used with CSS presentational elements such as style="" or inside the stylesheet itself.

Looking at the XHTML 1.0 Transitional DTD confirms this

Code: [Select]

<!ENTITY % Length "CDATA">
    <!-- nn for pixels or nn% for percentage length -->

<!ELEMENT table
     (caption?, (col*|colgroup*), thead?, tfoot?, (tbody+|tr+))>
<!ELEMENT caption  %Inline;>
<!ELEMENT thead    (tr)+>
<!ELEMENT tfoot    (tr)+>
<!ELEMENT tbody    (tr)+>
<!ELEMENT colgroup (col)*>
<!ELEMENT col      EMPTY>
<!ELEMENT tr       (th|td)+>
<!ELEMENT th       %Flow;>
<!ELEMENT td       %Flow;>

<!ATTLIST table
  %attrs;
  summary     %Text;         #IMPLIED
  width       %Length;       #IMPLIED
  border      %Pixels;       #IMPLIED
  frame       %TFrame;       #IMPLIED
  rules       %TRules;       #IMPLIED
  cellspacing %Length;       #IMPLIED
  cellpadding %Length;       #IMPLIED
  align       %TAlign;       #IMPLIED
  bgcolor     %Color;        #IMPLIED
  >


I only had to update themes.inc.php and minibrowser.php. Committed.