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

On Fri Apr 23, 2004 at 12:2404PM -0400, Daniel Convissor wrote: > I'm cc'ing pear-core on this. That IS the right list for the following, > right? I believe that pear-webmaster is more appropriate. > > - <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. Which browser do not understand the syntax? > 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. Makes sense. I've CCed Helgi Pormar, who provided the patch. -- - Martin Martin Jansen http://martinjansen.com/