Re: cvs: pearweb /include pear-format-html.php /public_html style.css

Hi: I'm cc'ing pear-core on this. That IS the right list for the following, right? On Fri, Apr 23, 2004 at 03:03:58PM -0000, Martin Jansen wrote: > mj Fri Apr 23 11:03:58 2004 EDT > > Modified files: > /pearweb/include pear-format-html.php > /pearweb/public_html style.css > Log: > * Improving XHTML validity > > # Patch by: dufuz ... snip ... > - <td align="right" valign="top" colspan="3" > height="1">&nbsp;</td> > + <td colspan="3" style="height: 1px; text-align: right; > vertical-align: top;">&nbsp;</td> I disagree with the direction this patch is going. Nearly all browsers can understand the old syntax. A limited set of browsers can understand the new syntax. Granted the number of people using browsers that can't understand the new syntax is small, there's no good reason to mess them up. Also, for long term flexibility and ease of design, I feel it's best to put a class name in the element itself and then specify the style in a style sheet rather than specifying the style directly in the element. Thanks for your consideration, --Dan -- T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y data intensive web and database programming http://www.AnalysisAndSolutions.com/ 4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409