Fix for 3784686, hidden links still active. Make sure text nodes exclude themselves...
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Sep 2004 18:22:56 +0000 (18:22 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Sep 2004 18:22:56 +0000 (18:22 +0000)
when hidden.

        * khtml/rendering/render_text.cpp:
        (RenderText::nodeAtPoint):

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

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

index ff1d580cebcca2f3a367977d957efb18ef764c32..8de73a9c97fe5c97b0a0d0a8277ed6c7e547ad43 100644 (file)
@@ -1,3 +1,11 @@
+2004-09-01  David Hyatt  <hyatt@apple.com>
+
+       Fix for 3784686, hidden links still active.  Make sure text nodes exclude themselves from nodeAtPoint checks
+       when hidden.
+       
+        * khtml/rendering/render_text.cpp:
+        (RenderText::nodeAtPoint):
+
 2004-08-31  David Hyatt  <hyatt@apple.com>
 
        Fixes for 3761411 and 3782201.  This patch makes two changes to percentage heights.  First it makes sure that
index a096e16f0b938715e36b116c56f428361887defb..de9fc38005e46681812648e82256a3f08a3b59d9 100644 (file)
@@ -574,6 +574,9 @@ bool RenderText::nodeAtPoint(NodeInfo& info, int _x, int _y, int _tx, int _ty,
 {
     assert(parent());
 
+    if (style()->visibility() == HIDDEN)
+        return inside;
+
     for (InlineTextBox *s = m_firstTextBox; s; s = s->nextTextBox()) {
         if((_y >=_ty + s->m_y) && (_y < _ty + s->m_y + s->height()) &&
            (_x >= _tx + s->m_x) && (_x <_tx + s->m_x + s->m_width) ) {