KDE Bug Tracking System
Home
Report New Wish or Bug
Query Existing Reports
First
Last
Prev
Next
No search results available
Search page
Bug
125185
:
Overlapping text on groups.google.com
P
roduct
:
konqueror
Co
m
ponent
:
khtml renderer
Status
:
RESOLVED
Resolution
:
FIXED
Target
:
---
Version
:
unspecified
Pr
i
ority
:
NOR
Severity
:
normal
V
otes
:
0
Description
:
Opened:
2006-04-08 19:25
Last Changed:
2006-04-14 02:27:29
Version: 3.5.2 (using KDE 3.5.2, Kubuntu Package 4:3.5.2-0ubuntu0 breezy) Compiler: Target: i486-linux-gnu OS: Linux (i686) release 2.6.12-10-386 Go to groups.google.com and view any message. In the box that shows 'From:' and 'Date:', there is some text: "show options", which is a link and "not yet rated". These two things overlap. Example link:
http://groups.google.com/group/linux.kernel/browse_thread/thread/b00f35cac273db1f/df37f24588be22aa#df37f24588be22aa
Comment
#1
Haris Kouzinopoulos 2006-04-08 22:09:56
Same here
Comment
#2
Thiago Macieira 2006-04-10 19:13:04
Confirmed on KDE 3.5.2 r522440. This is the code that breaks: <td><div style="position: absolute"><nobr><font color="#555555">Not yet rated</font></nobr></div> If you change that "position: absolute", it works.
Comment
#3
Germain Garand 2006-04-14 00:04:38
regressed in 3.5.0... likely from the inclusion in lineboxes of positioned children found in initial whitespace for #113527
Comment
#4
Germain Garand 2006-04-14 02:27:28
SVN commit 529640 by ggarand: Fix static Y position of positioned objects in initial whitespace (3.5 regression). we don't want to clear them to next line as there's no previous in-flow linebox. Change bogus/misleading comment. BUG: 125185 M +9 -3 bidi.cpp --- branches/KDE/3.5/kdelibs/khtml/rendering/bidi.cpp #529639:529640 @@ -792,12 +792,18 @@ if (bottomOfLine > m_height && bottomOfLine > m_overflowHeight) m_overflowHeight = bottomOfLine; + bool beforeContent = true; + // Now make sure we place replaced render objects correctly. for (BidiRun* r = sFirstBidiRun; r; r = r->nextRun) { - // Align positioned boxes with the top of the line box. This is - // a reasonable approximation of an appropriate y position. + + // For positioned placeholders, cache the static Y position an object with non-inline display would have. + // Either it is unchanged if it comes before any real linebox, or it must clear the current line (already accounted in m_height). + // This value will be picked up by position() if relevant. if (r->obj->isPositioned()) - r->box->setYPos(m_height); + r->box->setYPos( beforeContent && r->obj->isBox() ? static_cast<RenderBox*>(r->obj)->staticY() : m_height ); + else if (beforeContent) + beforeContent = false; // Position is used to properly position both replaced elements and // to update the static normal flow x/y of positioned elements.
P
latform
:
unspecified
O
S
:
Linux
K
eywords
:
People
Reporter
:
Amit Shah
Assigned To
:
Konqueror Developers
Related actions
View Bug Activity
Format For Printing
XML
Clone This Bug
Note
You need to
log in
before you can comment on or make changes to this bug.
Attachments
Add an attachment
(proposed patch, testcase, etc.)
Depends on
:
B
locks
:
Show dependency tree
-
Show dependency graph
First
Last
Prev
Next
No search results available
Search page
Actions
Reports
Requests
Reports
Bugs reported today
Bugs reported in the last 3 days
Bug reports with patches
Weekly Bug statistics
The most hated bugs
The most severe bugs
The most frequently reported bugs
The most wanted features
Junior Jobs
Report ownership counts and charts
My Account
New Account
Log In