> ey guys, I looked at DynAPI today and saw that it wasn't xHTML compliant,
> so I made a small file that documents incompatibilities with xHTML on the 
> 2.53 release.

Nice work! I'd vote for adopting most of the changes.

> xHTML Analysis of DynAPI v2.53
> Author: Anarchos
> Date: 6-15-01
> E-mail: [EMAIL PROTECTED]
> Comments: word-wrap is off for line numbers.
> */
> 
> src/
>         dynapi.js
>                 187 - type="text/javascript" required for <script>
>                 189 - type="text/javscript" required for <script>

There's another issue with the include statement. It uses
language="Javascript1.2".
I just had some problems in my own library with prescribing a specific
Javascript version. If you're interested look into
http://bugzilla.mozilla.org/show_bug.cgi?id=86166

Once used for preventing Netscape 3 from loading a Javascript file and
producing syntax errors, the drawbacks of this technique far outweigh
the benefits today. I'd propose to cut the "1.2".

>  161 - attribute requires quotes (left)
>                         162 - attribute requires quotes (top)
>                         163 - attribute requires quotes (width)
>                         164 - attribute requires quotes (height)
>                         167 - attribute requires quotes (zIndex)
>                         167 - attribute should be lowercase (zIndex)

Netscape4 is no xhtml browser. So this will never be a problem but
surely it would be better style.

And there's another tiny bug hiding...
  if (this.z) s+=' zIndex='+this.z;
should be
  if (this.z) s+=' z-index='+this.z;
z-index (not zIndex) is the correct attribute name, even for Netscape 4.

>  label.js
>                         64 - element deprecated (font)

I don't think we can do something about this.

Stephan


_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dynapi-dev

Reply via email to