170650 2008-09-08 01:14:26 +0000 [testcase] Broken website layout - http://pollycoke.net 2008-09-15 04:31:20 +0000 1 1 2 Applications konqueror khtml 4.1.0 Compiled Sources Unspecified RESOLVED FIXED NOR normal --- 1 luigi.toscano konqueror-bugs-null frank78ac germain 0 oldest_to_newest 644586 0 luigi.toscano 2008-09-08 01:14:26 +0000 Version: (using KDE 4.1.1) Installed from: Compiled From Sources The layout of http://pollycoke.net/, an italian blog wordpress-based, is broken in KDE 4.x while it renders correctly in 3.5.x. 644840 1 germain 2008-09-09 00:35:24 +0000 when I disable JS the rendering is fine, so might be a script misbehaving.. not sure why it would not affect 3.5 in the same way though. 645381 2 27356 frank78ac 2008-09-10 21:43:54 +0000 Created attachment 27356 Reduced test case This reduced test case still shows the strange behaviour for me - some text is shown outside the box in 4.1.1, but not in 3.5.10. I can now reproduce it without JS although I'm pretty sure that removing the links to the ".js" files did not work at an earlier stage in the reduction. It's also funny that the buggy behaviour depends on the "<!DOCTYPE..." declaration. Remove it or change "XHTML Strict" to "XHTML Transitional", and the bug goes away. Replacing "XHTML Strict" by "HTML Strict" keeps the buggy behaviour though. 645671 3 germain 2008-09-11 22:16:24 +0000 thanks for the nice testcase! so the 'bug' is that we try to do stricter, more standard HTML parsing in 4.x in standard mode. Apparently, this is way too optimistic for the web. I'll revert to more loosy parsing in all modes. 646521 4 germain 2008-09-15 04:31:20 +0000 SVN commit 861054 by ggarand: .doing strict dtd checking for inlines in strict mode turns out to be too optimistic. It breaks sites. .no popBlock for dl/dt errors. It doesn't match what other browsers do. BUG: 170650 M +2 -5 dtd.cpp M +0 -8 htmlparser.cpp --- trunk/KDE/kdelibs/khtml/html/dtd.cpp #861053:861054 @@ -474,14 +474,11 @@ case ID_FONT: case ID_LEGEND: case ID_Q: + case ID_A: case ID_NOBR: case ID_WBR: // %inline * - return check_inline(childID, strict) || - (!strict && check_block(childID, strict)); - case ID_A: - // A: %inline * (but even strict sites expect %flow) - return check_flow(childID, strict); + return check_inline(childID, strict) || check_block(childID, strict); case ID_P: // P: %inline * return check_inline(childID, strict) || --- trunk/KDE/kdelibs/khtml/html/htmlparser.cpp #861053:861054 @@ -801,14 +801,6 @@ insertNode(e); handled = true; break; - case ID_DL: - popBlock(ID_DL); - handled = true; - break; - case ID_DT: - popBlock(ID_DT); - handled = true; - break; case ID_FORM: popBlock(ID_FORM); handled = true; 27356 2008-09-10 21:43:54 +0000 2008-09-10 21:43:54 +0000 Reduced test case testcase.html text/html 540 frank78ac PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjAgU3RyaWN0Ly9FTiIg Imh0dHA6Ly93d3cudzMub3JnL1RSL3hodG1sMS9EVEQveGh0bWwxLXN0cmljdC5kdGQiPgo8aHRt bCB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRtbCI+CjxoZWFkPgo8dGl0bGU+VGVz dCBjYXNlIGZvciBidWcgMTcwNjUwPC90aXRsZT4KPG1ldGEgaHR0cC1lcXVpdj0iY29udGVudC10 eXBlIiBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9VVRGLTgiIC8+CjwvaGVhZD4KPGJvZHk+ CiAgPGRpdiBzdHlsZT0iYm9yZGVyOjVweCBzb2xpZCAjZGRlOyI+CiAgICA8ZW0gc3R5bGU9ImRp c3BsYXk6YmxvY2s7Ij4KICAgICAgPGRpdj5UaGlzIHRleHQgYXBwZWFycyBhbHdheXMgaW5zaWRl IHRoZSBib3guPC9kaXY+CiAgICA8L2VtPgogICAgVGhpcyB0ZXh0IGFwcGVhcnMgaW5zaWRlIHRo ZSBib3ggaW4gRmlyZWZveCAzIGFuZCBLb25xdWVyb3IgMy41LjEwLCBidXQgbm90IGluIDQuMS4x LgogIDwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4K