The Samba-Bugzilla – Bug 4032
"show diffs" css too broken for Opera and IE
Last modified: 2007-10-04 05:59:40 UTC
in IE and Opera 9 (both tested in Windows) just can't handle the part of css which is for displaying the patch sources if you click on "show diffs".
(or any "show diffs" link on the recent checkins site)
Doesn' work with Konqueror, Opera, IE ... only Mozilla-base browsers work.
I just tested IE7 on Windows XP+SP2 and it cannot even display the Build Farm home page (http://build.samba.org). The build-farm script returns a filename of "build_farm_org" which leads IE7 to display a dialog box indicating that the file has an "unknown file type". If you save the file to disk and rename it so it ends in ".html", then IE7 can access the web page; however, it is unable to display it properly. Some of the items on the web page overlap each other. So basically, the build farm is inaccessible from IE7. I didn't try older versions of IE.
Paul, can you please open a new bug for that problem as this is not relates to this CSS bug ...
Created attachment 2935 [details]
fix for it
if web browsers don't see a valid end of a <!--foo--> html comment they don't know when the html code stops babbling around :-)
now checked in by metze, thanks!
Horay! IE7 works now! I can load the home page and click on links and the pages load properly and look fine. Many thanks. PG
you're welcome. The problem you had with IE was that the page was delivered with Content-Type x-perl instead of text/html.