#23203 [Opn->Bgs]: phpinf() css/xhtml error

ID: 23203 Updated by: sniper@php.net Reported By: cjb at filmnet dot com dot au -Status: Open +Status: Bogus Bug Type: PHP options/info functions Operating System: Mac OS X PHP Version: 4.3.0 New Comment: Get working browser. Previous Comments: ------------------------------------------------------------------------ [2003-04-19 14:23:37] ken at tossell dot net It really isn't necessary to comment this out. The diff below removes the <!-- (ext/standard/info.c) --- info.c-old Sat Apr 19 15:21:04 2003 +++ info.c Sat Apr 19 15:19:56 2003 @@ -183,9 +183,9 @@ */ void php_info_print_style(void) { - php_printf("<style type=\"text/css\"><!--\n"); + php_printf("<style type=\"text/css\">\n"); php_info_print_css(); - php_printf("//--></style>\n"); + php_printf("</style>\n"); } /* }}} */ ------------------------------------------------------------------------ [2003-04-14 09:57:34] cjb at filmnet dot com dot au The phpinfo() function generates a web page written to the XHTML 1.0 transitional DTD. This page incorrectly uses the HTML technique of hiding CSS styles from "old" browsers useing the "<!-- -->" style comments. The XHTML spec allows user agents to ignore CSS between such comments <http://www.w3.org/TR/xhtml1/#C_4> as my broswer does. The end results is I don't get styled markup of php info in my browser... which is striclty applying the XHTML 1.0 spec. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=23203&edit=1