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

On Fri, 2004-04-23 at 16:48, Martin Jansen wrote: > 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. With great respect for you Daniel ... You're so very much wrong ... I've been making websites now that are valid in IE6 5.5 and 5.0 as well as those that are based on the Gecko engine (and even tested those sites on Opera) and everyone of these browser understand style="" ... Also all the css in the style="" you pasted are CSS1 and should be understood by down to IE4 Those that are using browsers that don't understand CSS either have very very very outdated browsers. And I am trying to make the site XHTML valid, so why should we be keeping onto this that aren't XHTML valid ? If you can mention any browsers that can't support CSS for some reason then I'll take that to consideration, but if you haven't been seeing what's happening then most of the people in the world use infact IE 5.0+, and IE6 well the most widly used browser.