Summary: | [regression] Mistake in picture placing on page | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Martin Reichel <martin> |
Component: | khtml renderer | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | gassauer |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: |
minimal test case
patch shows rendering problem |
Description
Martin Reichel
2006-06-07 16:54:19 UTC
*** Bug 130066 has been marked as a duplicate of this bug. *** 3.5.x regression. Created attachment 16836 [details]
minimal test case
whitespace (that should be discarded) issue.
Interestingly, it only happens with absolutely positioned blocks having both
non-static x and y..
Created attachment 16838 [details]
patch
WebCore has this bit, and it looks all fine to me..
similar problem here ? http://wiki.gnutelephony.org/index.php/Main_Page re #5: no obviously not. your url is rendered just as designed. cf. http://wiki.gnutelephony.org//skins/monobook/KHTMLFixes.css probably intended to work around some prehistoric bug. Created attachment 16850 [details]
shows rendering problem
firefox handles this page well , konqueror not.
re #7: Ferdinand, please read carefully the messages. I told you the site intentionally breaks its rendering for khtml-based engines based on JavaScript sniffing. Just have them remove the bogus "KHTMLFixes.css" css file. SVN commit 557104 by ggarand: skip out of flow objects when looking for a significant previous renderer in rendererIsNeeded. BUG:128760 M +2 -0 dom_textimpl.cpp --- branches/KDE/3.5/kdelibs/khtml/xml/dom_textimpl.cpp #557103:557104 @@ -389,6 +389,8 @@ } RenderObject *first = par->firstChild(); + while (first && first->isFloatingOrPositioned()) + first = first->nextSibling(); RenderObject *next = nextRenderer(); if (!first || next == first) { // Whitespace at the start of a block just goes away. Don't even |