Created an attachment (id=3699) [details]
this shows the erroneous vertical space

Created an attachment (id=3700) [details]
mozilla shows the page correctly (same as konqueror 3.1.4)

Bug confirmed here.

Version: 3.1.94
OS: Linux Gentoo
Compiler: gcc (GCC) 3.2.3 20030422 (Gentoo Linux 1.4 3.2.3-r3, propolice)


What seems to trigger the bug is an empty table cell (<TR><TD><BR></TD></TR>) above the form. If you remove this code altogether, Konqueror's rendering is the same as Mozilla's (both a little off due to the cell's removal). Replacing this code with <TR></TR> is however sufficient to trigger the bug.

Created an attachment (id=3708) [details]
Test case for the bug

Here is a test case for the bug. In Mozilla, it shows two lines of text at the
top of the page. In Konqueror, the first line is at the top of the page and the
second one much further down.

As of release 3.2.0rc1, this bug is still not solved.

This still does not work with 3.2RC
Cannot read my mail with my companies web-interface :-(

Thx for all the hard work!

*** Bug 72689 has been marked as a duplicate of this bug. ***

Is this bug related to (dup of) bug #22657?

Cheers Jo

*** Bug 92059 has been marked as a duplicate of this bug. ***

*** Bug 75153 has been marked as a duplicate of this bug. ***

*** Bug 81354 has been marked as a duplicate of this bug. ***

CVS commit by carewolf: 

Calculate percentage height from containing block and not viewport.
Fixes most of the bugs that have been incorrectly closed as duplicates of 70418, 
and the email composer in gmail.com. 70418 itself is a different bug though.
BUG: 75153, 81354, 72689, 92059
CCBUG: 70418


  M +1 -1      render_box.h   1.83
  M +2 -33     render_table.cpp   1.263


--- kdelibs/khtml/rendering/render_box.h  #1.82:1.83
@@ -114,5 +114,5 @@ public:
     RenderBlock* createAnonymousBlock();
 
-private:
+protected:
     int calcBoxWidth(int w) const;
     int calcBoxHeight(int h) const;

--- kdelibs/khtml/rendering/render_table.cpp  #1.262:1.263
@@ -296,5 +296,4 @@ void RenderTable::layout()
     m_height = oldHeight;
 
-    // html tables with percent height are relative to view
     Length h = style()->height();
     int th = -(bpTop + bpBottom); // Tables size as though CSS height includes border/padding.
@@ -303,36 +302,6 @@ void RenderTable::layout()
     else if (h.isFixed())
         th += h.value();
-    else if (h.isPercent()) {
-        RenderObject* c = containingBlock();
-        for ( ;
-            !c->isCanvas() && !c->isBody() && !c->isTableCell() &&
-                  !c->isPositioned() && c->isFloating();
-             c = c->containingBlock()) {
-            Length ch = c->style()->height();
-            if (ch.isFixed()) {
-                th += h.width(ch.value());
-                break;
-            }
-        }
-
-        if (c->isTableCell()) {
-            RenderTableCell* cell = static_cast<RenderTableCell*>(c);
-            int cellHeight = cell->cellPercentageHeight();
-            if (cellHeight)
-                th += h.width(cellHeight);
-        }
-        else  {
-            Length ch = c->style()->height();
-            if (ch.isFixed())
-                th += h.width(ch.value());
-            else {
-                // we need to substract out the margins of this block. -dwh
-                th += h.width(viewRect().height() - c->marginBottom() - c->marginTop());
-                // not really, but this way the view height change
-                // gets propagated correctly
-                setOverhangingContents();
-            }
-        }
-    }
+    else if (h.isPercent())
+        th += calcPercentageHeight(h);
 
     // layout rows

CVS commit by carewolf: 

Improve distribution of excess height.
- Do not distribute to empty variable rows to start with
- Add extra distribution step to handle table with _only_ empty variable rows
- Add extra distribution step to handle round-off excess
BUG: 70418
PS: Please note that we still don't render mail.beteor.nl/servlet/webacc like
IE does, but we not have fewer problems on that page than Mozilla does.


  M +18 -2     ChangeLog   1.334
  M +47 -12    rendering/render_table.cpp   1.266
  M +1 -0      rendering/render_table.h   1.108