Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix offsetTop/offsetLeft to not be confused by table borders. Bugs

Browse Source 
163923 and 119167, r=sicking, sr=jst


git-svn-id: svn://10.0.0.236/trunk@129196 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
bzbarsky%mit.edu 2002-09-11 01:58:59 +00:00
parent 18e737b843
commit 7b4eaeec7f
1 changed files with 20 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
mozilla/content/html/content/src/nsGenericHTMLElement.cpp
View File

@ -735,19 +735,31 @@ nsGenericHTMLElement::GetOffsetRect(nsRect& aRect,
// And subtract out the border for the parent
if (parent) {
border = nsnull;
PRBool includeBorder = PR_TRUE; // set to false if border-box sizing is used
const nsStylePosition* position = nsnull;
parent->GetStyleData(eStyleStruct_Position, (const nsStyleStruct*&)position);
if (position && position->mBoxSizing == NS_STYLE_BOX_SIZING_BORDER) {
includeBorder = PR_FALSE;
}
if (includeBorder) {
border = nsnull;
parent->GetStyleData(eStyleStruct_Border, (const nsStyleStruct*&)border);
if (border) {
if (eStyleUnit_Coord == border->mBorder.GetLeftUnit()) {
origin.x -= border->mBorder.GetLeft(coord).GetCoordValue();
}
if (eStyleUnit_Coord == border->mBorder.GetTopUnit()) {
origin.y -= border->mBorder.GetTop(coord).GetCoordValue();
parent->GetStyleData(eStyleStruct_Border, (const nsStyleStruct*&)border);
if (border) {
if (eStyleUnit_Coord == border->mBorder.GetLeftUnit()) {
origin.x -= border->mBorder.GetLeft(coord).GetCoordValue();
}
if (eStyleUnit_Coord == border->mBorder.GetTopUnit()) {
origin.y -= border->mBorder.GetTop(coord).GetCoordValue();
}
}
}
}
// XXX We should really consider subtracting out padding for
// content-box sizing, but we should see what IE does....
// Get the scale from that Presentation Context
float scale;
context->GetTwipsToPixels(&scale);