Bug 85150 - [test case] floating elements don't respect bottom margin of surrounding elements
Summary: [test case] floating elements don't respect bottom margin of surrounding elem...
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-14 08:25 UTC by Yves Glodt
Modified: 2004-08-05 19:58 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
testcase (2.30 KB, text/html)
2004-07-14 08:26 UTC, Yves Glodt
Details
reduced test case (383 bytes, text/html)
2004-07-19 07:46 UTC, Stephan Kulow
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yves Glodt 2004-07-14 08:25:58 UTC
Version:            (using KDE KDE 3.2.3)
Installed from:    Debian testing/unstable Packages
OS:                Linux

Hi,

I have a problem with a simple html-layout that is realized through
css. I attached a testcase.

The problem is that the first group of the 4 divs "4 3 2 1" correctly
touches the div below, but every following group does not. It seems like 
the "margin-bottom:10px" rule only gets applied on the first group.

Maybe it shows a problem in khtml, but I'm not sure about that, it could
also be a problem in the html/css. (but opera, gecko and ie display it
fine)

The html for each block of the 8 divs is identic.

I suck at explaining, so better look at the result in khtml 3.2.3 to see
the problem... ;-)


(created a bugreport and attached the testcase as requested by Germain
in this thread: http://lists.kde.org/?l=kfm-devel&m=108974839709306&w=2 )
Comment 1 Yves Glodt 2004-07-14 08:26:32 UTC
Created attachment 6673 [details]
testcase
Comment 2 Stephan Kulow 2004-07-19 07:46:28 UTC
Created attachment 6724 [details]
reduced test case

this test case shows a bit better what's going on.
Comment 3 Germain Garand 2004-08-05 02:32:38 UTC
CVS commit by ggarand: 

- when positionning floats, use the margin tracking variables, and not the prevFlow - there might be
  self-collapsing blocks in-between (#85150).

- fix Marquee initial layout.

- calcMinMaxWidth fixes (webcore/fast/{block/positioning/060.html,text/whitespace/025.html})

CCMAIL: 85150-done@bugs.kde.org, l.savernik@aon.at
									


  M +16 -0     ChangeLog   1.275
  M +25 -17    rendering/render_block.cpp   1.38
  M +0 -1      rendering/render_object.cpp   1.267
  M +3 -3      rendering/render_object.h   1.192
  M +3 -1      xml/dom_elementimpl.cpp   1.200



Comment 4 Yves Glodt 2004-08-05 19:58:26 UTC
Merci Germain, t'es une bête!