Bug 138146

Summary:	CSS "display: table" formats improperly
Product:	[Applications] konqueror	Reporter:	Maciej J . Woloszyk <matofesi>
Component:	khtml renderer	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	d-frog
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Maciej J . Woloszyk 2006-11-30 12:43:23 UTC

Version:           3.5.5 (using KDE 3.5.5, Gentoo)
Compiler:          Target: i686-pc-linux-gnu
OS:                Linux (i686) release 2.6.16-gentoo-r13

When opening http://www.merlin.com.pl/ you get messed-up display - the text under the orange "Klienci Merlina polecaja" (and also below that) is put on top (or maybe it's below) of images. It looks that the problem is in http://www.merlin.com.pl/css/new/layout4.css ".rightSection" class declaration - there is "display: table;" there and it seems to mess up the rendering - when I saved page and layout and removed this declaration text is placed a bit off, but it's not rendered on the images. What's more interesting is that sometimes when the page loads everything looks fine until you move mouse and then the text moves to the left.

Comment 1 Tommi Tervo 2006-11-30 12:54:17 UTC

Could you make a small test case?

Comment 2 Maciej J . Woloszyk 2006-11-30 14:12:01 UTC

Hmmm... I tried to make the smallest test case showing the problem and what I fount is that the problem is probably somewhere in JavaScript.

Test case can be seen here: http://www.janussa.pl/138146-testcase/

It should load and look like this: http://www.janussa.pl/138146-testcase/snap1.jpg

Then when you move your mouse over the images, links etc. texts suddenly jump to the left and then it looks like this:

http://www.janussa.pl/138146-testcase/snap2.jpg

Offender seems to be somewhere in the file http://www.janussa.pl/138146-testcase/skrypt2.js (which is the original file cut down to the minimum).

m.

Comment 3 Allan Sandfeld 2006-11-30 14:41:08 UTC

I get the same problem even without the javascript. 

Something makes the relayout on mouseover act differently.

Comment 4 Maciej J . Woloszyk 2006-11-30 14:47:49 UTC

Ok. Didn't catched that... When cuting down the code I thought that removing scripts solved the problem... Now I can see it is something with styles - it acts the same without script or even with js disabled. Here is the test case without script: http://www.janussa.pl/138146-testcase/index1.html , and here: http://www.janussa.pl/138146-testcase/index2.html is the same without "display: table" in "rightSection" class.

m.

Comment 5 Maksim Orlovich 2006-11-30 15:18:58 UTC

Very weird --- for me your reduced versions render fine, but the website doesn't....

Comment 6 Maciej J . Woloszyk 2006-11-30 15:22:13 UTC

Problem is that this happens in kin of random fashion - when creating testcase I have a lot of problems to identify what is absolutely necessary to show the problem. Try loading testcase a few times and move mouse pointer over it - it should show up.

m.

Comment 7 Maksim Orlovich 2006-11-30 15:26:19 UTC

Tommi pointed out a way for me to reproduce it --- clicking on the image... Still, it just happens on the original.... Let's see if VG says anything..

Comment 8 Allan Sandfeld 2006-11-30 16:45:17 UTC

I never see it when first loaded. Only when after hovering a link or opening DOM viewer

Comment 9 Germain Garand 2007-01-02 10:29:01 UTC

SVN commit 618830 by ggarand:

table objects always flowAroundFloats
BUG: 138146


 M  +1 -0      render_table.cpp  


--- branches/KDE/3.5/kdelibs/khtml/rendering/render_table.cpp #618829:618830
@@ -84,6 +84,7 @@
     ETableLayout oldTableLayout = style() ? style()->tableLayout() : TAUTO;
     if ( _style->display() == INLINE ) _style->setDisplay( INLINE_TABLE );
     if ( _style->display() != INLINE_TABLE ) _style->setDisplay(TABLE);
+    if ( !_style->flowAroundFloats() ) _style->setFlowAroundFloats(true);
     RenderBlock::setStyle(_style);
 
     // init RenderObject attributes

Comment 10 Germain Garand 2007-01-25 21:06:20 UTC

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