Re: cvs: php4 /ext/standard config.m4 css.c css.h info.c /main php_ini.c /sapi/aolserver aolserver.c /sapi/apache php_apache.c

Colin Viebrock <colin@easydns.com> wrote: > derick@php.net wrote: >> "render nicely" is something else than "total unreadable" to me. Did >> you >> have a look at it with Netscape Navigator 4? Please fix something for >> it, as it's pretty horrible now. > > Can you send me a screen grab? I did test it under NS 4.7 (Win), and I > think you're exaggerating a bit, because it's hardly unreadable. With Netscape 4.7x on Linux and Windows, I see a lot of vertical whitespace in some places, e.g. directly after "PHP Core" there's a screen full of nothing. > I guess my other question is how far back are we willing to care about > browser compatability? Netscape 4.7 is now 3 years old and accounts for > less 5% of the traffic we've seen at easyDNS in the last year (I don't > think the stats program at www.php.net tracks browsers unfortunately). At work, we still have to use 4.7x on the Unix machines in the intranet. There's no internet connection, so these won't ever show up in any browser stats. The patch below addresses that whitespace problem. According to <http://pixels.pixelpark.com/~koch/hide_css_from_browsers/media/>, it only affects Netscape 4.x, and the W3C CSS validator has no problems with it. I also tested the resulting page on IE5/Win and Mozilla 1.2a and Opera 6.0 on Linux. Can you please apply it? Regards... Michael --- php4/ext/standard/css.c~ Thu Sep 19 23:57:24 2002 +++ php4/ext/standard/css.c Thu Sep 26 00:26:28 2002 @@ -58,9 +58,13 @@ PUTS("}\n"); PUTS("td, th {"); PUTS("vertical-align: baseline;"); - PUTS("padding: 3px;"); PUTS("border: 1px solid #000000;"); PUTS("}\n"); + PUTS("@media all { td, th {"); + PUTS("vertical-align: baseline;"); + PUTS("padding: 3px;"); + PUTS("border: 1px solid #000000;"); + PUTS("} }\n"); PUTS("h1 {"); PUTS("text-align: center;"); PUTS("font-size: 160%;");