Bug 76449

Summary: [test case] regression on table rendering - space left between rows
Product: [Applications] konqueror Reporter: Mark <mveinot>
Component: khtml rendererAssignee: Konqueror Bugs <konqueror-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: test case

Description Mark 2004-02-29 21:04:02 UTC 
Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          3.2.2 
OS:          Linux

my personal index page on my server doesn't render correctly.

This is a regression that occurred in HEAD within a week after the release of KDE 3.2.

http://mvgrafx.dyn.ca

There is a space between the top <tr> and the next as well as several spacing problems at the bottom.

This page renders correctly in KDE 3.2 and in all other major browsers.
Comment 1 Stephan Kulow 2004-02-29 21:59:47 UTC 
Created attachment 4944 [details]
test case
Comment 2 Dirk Mueller 2004-02-29 23:04:38 UTC 
CVS commit by mueller: 

treat xhtml transitional as transitional
CCMAIL: 76449-done@bugs.kde.org


  M +3 -0      ChangeLog   1.228
  M +1 -1      html/html_documentimpl.cpp   1.160


--- kdelibs/khtml/ChangeLog  #1.227:1.228
@@ -1,4 +1,7 @@
 2004-02-29  Dirk Mueller  <mueller@kde.org>
 
+        * html/html_documentimpl.cpp (determineParseMode): treat xhtml transitional
+        that way (#76449).
+
         * rendering/bidi.cpp (findNextLineBreak): make sure we move
         replaced elements, that don't fit on a line below the floats (#74609).

--- kdelibs/khtml/html/html_documentimpl.cpp  #1.159:1.160
@@ -420,5 +420,5 @@ void HTMLDocumentImpl::determineParseMod
 
         if ( hMode == XHtml )
-            pMode = Strict;
+            pMode = publicId;
     }
     // kdDebug() << "DocumentImpl::determineParseMode: publicId =" << publicId << " systemId = " << systemId << endl;
Comment 3 Mark 2004-03-01 00:59:25 UTC 
Wow, guys - that was blindingly fast!

The KDE team is awesome :)