Reviewed by Maciej.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Jul 2004 21:42:11 +0000 (21:42 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Jul 2004 21:42:11 +0000 (21:42 +0000)
        - fixed <rdar://problem/3691569> REGRESSION (142): cmd-shift-clicking on a link
        now also extends selection (even if there wasn't one before)

        * khtml/khtml_part.cpp:
        (KHTMLPart::handleMousePressEventSingleClick):
        Uh, duh. Got a ! wrong last time somehow, so it did even more the wrong thing
        than ever. I did test this before, so my best guess is an accidental undo or
        something before committing.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/khtml_part.cpp

index 5ffac1ea2c1ac68c249df6dcd745747397d7cc02..c9ce4c1c90071fb8b0096ef81593f651642e3d00 100644 (file)
@@ -1,3 +1,16 @@
+2004-07-23  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Maciej.
+
+        - fixed <rdar://problem/3691569> REGRESSION (142): cmd-shift-clicking on a link 
+        now also extends selection (even if there wasn't one before)
+
+        * khtml/khtml_part.cpp:
+        (KHTMLPart::handleMousePressEventSingleClick):
+        Uh, duh. Got a ! wrong last time somehow, so it did even more the wrong thing
+        than ever. I did test this before, so my best guess is an accidental undo or
+        something before committing.
+
 2004-07-23  David Hyatt  <hyatt@apple.com>
 
        Fix for 3735084 and 3737209, when you can find no line boxes in your previous sibling, the line box to dirty
index 5a41a764b9f5eaef1579158a957f53a91891262c..e5d7babd84a9733b3cb4072c3e947a55b33fe14f 100644 (file)
@@ -4402,7 +4402,7 @@ void KHTMLPart::handleMousePressEventSingleClick(khtml::MousePressEvent *event)
         if (!innerNode.isNull() && innerNode.handle()->renderer() &&
             innerNode.handle()->renderer()->shouldSelect()) {
             // Extend the selection if the Shift key is down, unless the click is in a link.
-            bool extendSelection = (mouse->state() & ShiftButton) && (!event->url().isNull());
+            bool extendSelection = (mouse->state() & ShiftButton) && (event->url().isNull());
 
             // Don't restart the selection when the mouse is pressed on an
             // existing selection so we can allow for text dragging.