82266 2004-05-26 20:31:50 +0000 [test case] Caption width is too short 2007-01-02 11:51:06 +0000 1 1 2 Applications konqueror khtml renderer unspecified unspecified Linux RESOLVED FIXED NOR normal --- 1 mattias konqueror-bugs-null hadmut lex.lists 0 oldest_to_newest 236341 0 mattias 2004-05-26 20:31:50 +0000 Version: (using KDE KDE 3.2.1) Installed from: Unlisted Binary Package OS: Linux The caption is not rendered over the entire table. Check out this testcase with konq and mozilla http://shayol-ghul.csbnet.se/~mattias/sadIII/caption3.html Screenshots below: http://shayol-ghul.csbnet.se/~mattias/sadIII/screen1.png http://shayol-ghul.csbnet.se/~mattias/sadIII/screen2.png 301630 1 kde 2005-01-10 15:42:00 +0000 Webpage is no longer active. 301877 2 mattias 2005-01-11 12:34:43 +0000 Sorry. Apache daemon went down. Webpage problem fixed. 335851 3 lex.lists 2005-04-19 14:47:40 +0000 Site couldn't be visited 2005-April-19 352446 4 lex.lists 2005-06-22 02:49:37 +0000 Mattias, I'm not 100% what the problem is. Can you please clarify what the problem is? 352474 5 mattias 2005-06-22 08:41:55 +0000 The problem is that the caption isn't rendered over the entire table. If you look at the right border, where the table and caption meet, you see that the caption is shorter than the rest of the table. 356945 6 lex.lists 2005-07-11 04:43:32 +0000 Mattias, the above links are currently broken. Please consider creating attachments of those files so that your bug can be worked on regardless of the status of your web server. 362439 7 12060 tommi.tervo 2005-08-03 12:55:26 +0000 Created attachment 12060 test case Caption is two pixels narrower than table (3.5-branch) 423377 8 tommi.tervo 2006-03-20 09:12:35 +0000 *** Bug 123952 has been marked as a duplicate of this bug. *** 478051 9 kde 2006-10-18 22:00:54 +0000 SVN commit 596875 by carewolf: captions are independent of tables border and padding BUG: 82266 M +4 -0 render_box.cpp --- branches/KDE/3.5/kdelibs/khtml/rendering/render_box.cpp #596874:596875 @@ -707,6 +707,10 @@ } RenderBlock* cb = containingBlock(); + if (isRenderBlock() && cb->isTable()) { + //captions are not affected by table border or padding + return cb->width(); + } if (usesLineWidth()) return cb->lineWidth(m_y); else 478878 10 germain 2006-10-22 02:44:42 +0000 + if (isRenderBlock() && cb->isTable()) { + //captions are not affected by table border or padding this doesn't screen out table cells, or I'm missing something? 497557 11 germain 2007-01-02 11:51:06 +0000 SVN commit 618852 by ggarand: don't let other elements go in there e.g. <table border=5><tr><td style=position:relative;left:100%>X CCBUG:82266 M +1 -1 render_box.cpp M +1 -0 render_table.h --- trunk/KDE/kdelibs/khtml/rendering/render_box.cpp #618851:618852 @@ -707,7 +707,7 @@ } RenderBlock* cb = containingBlock(); - if (isRenderBlock() && cb->isTable()) { + if (isRenderBlock() && cb->isTable() && static_cast<RenderTable*>(cb)->caption() == this) { //captions are not affected by table border or padding return cb->width(); } --- trunk/KDE/kdelibs/khtml/rendering/render_table.h #618851:618852 @@ -162,6 +162,7 @@ CollapsedBorderValue* currentBorderStyle() { return m_currentBorder; } RenderTableSection *firstBodySection() const { return firstBody; } + RenderFlow* caption() const { return tCaption; } protected: 12060 2005-08-03 12:55:26 +0000 2005-08-03 12:55:26 +0000 test case caption3.html text/html 333 tommi.tervo CgoKPGh0bWw+Cgo8aGVhZD4KCTx0aXRsZT5DYXB0aW9udGVzdDwvdGl0bGU+Cgk8bGluayByZWw9 InN0eWxlc2hlZXQiIGhyZWY9Imh0dHA6Ly9zaGF5b2wtZ2h1bC5jc2JuZXQuc2Uvfm1hdHRpYXMv ZGVmYXVsdC5jc3MiIC8+CjwvaGVhZD4KCjxib2R5PgoKPGRpdiBjbGFzcz0nRm9ybVZpZXcnPgo8 dGFibGUgY2xhc3M9J0Zvcm1WaWV3Jz4KPGNhcHRpb24+UmVuZGVyZWQgaW5jb3JyZWN0bHk8L2Nh cHRpb24+Cjx0cj4KPHRkPkZpbGxvdXRmaWxsb3V0ZmlsbG91dDwvdGQ+Cjx0ZD5GaWxsb3V0Zmls bG91dDwvdGQ+CjwvdHI+CjwvdGFibGU+CjwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4K