Hit testing in table cells with top/bottom space from vertical alignment didn't...
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Nov 2004 08:22:14 +0000 (08:22 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Nov 2004 08:22:14 +0000 (08:22 +0000)
super-secret yPos() lie that table cells do.  Use m_y instead of yPos().

        * khtml/rendering/render_block.cpp:
        (khtml::RenderBlock::nodeAtPoint):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8076 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/rendering/render_block.cpp

index bf055fc..0df78e0 100644 (file)
@@ -1,3 +1,11 @@
+2004-11-23  David Hyatt  <hyatt@apple.com>
+
+       Hit testing in table cells with top/bottom space from vertical alignment didn't work.  I forgot about the
+       super-secret yPos() lie that table cells do.  Use m_y instead of yPos().
+
+        * khtml/rendering/render_block.cpp:
+        (khtml::RenderBlock::nodeAtPoint):
+
 2004-11-22  David Hyatt  <hyatt@apple.com>
 
        Make sure you can use document.createElement to make a <canvas> element.
index ccfbc39..1c024c0 100644 (file)
@@ -2337,8 +2337,8 @@ bool RenderBlock::nodeAtPoint(NodeInfo& info, int _x, int _y, int _tx, int _ty,
 {
     bool inlineFlow = isInlineFlow();
 
-    int tx = _tx + xPos();
-    int ty = _ty + yPos();
+    int tx = _tx + m_x;
+    int ty = _ty + m_y;
 
     if (!inlineFlow && !isRoot()) {
         // Check if we need to do anything at all with this block.