Bug 76449 - [test case] regression on table rendering - space left between rows
Summary: [test case] regression on table rendering - space left between rows
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-02-29 21:04 UTC by Mark
Modified: 2004-03-01 00:59 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
test case (486 bytes, text/html)
2004-02-29 21:59 UTC, Stephan Kulow
Details

Note You need to log in before you can comment on or make changes to this bug.
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 :)