Bug 190255 - mousedown event is cancelable (W3C)
Summary: mousedown event is cancelable (W3C)
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: kjs (show other bugs)
Version: 4.2.2
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-21 15:00 UTC by lunter
Modified: 2009-04-23 17:39 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
source (1.15 KB, text/html)
2009-04-21 15:00 UTC, lunter 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description lunter 2009-04-21 15:00:26 UTC 
Version:           4.2.2 (using KDE 4.2.2)
OS:                MS Windows
Installed from:    MS Windows

W3C Documentation: mousedown event is cancelable
Konqueror: mousedown event is not cancelable

Run attachment.
Comment 1 lunter 2009-04-21 15:00:52 UTC 
Created attachment 32984 [details]
source
Comment 3 Maksim Orlovich 2009-04-21 15:36:13 UTC 
Most likely (can't test right now):
--- khtmlview.cpp       (revision 956244)
+++ khtmlview.cpp       (working copy)
@@ -3756,7 +3756,7 @@
         }
         me->deref();

-        if (eventId == EventImpl::MOUSEDOWN_EVENT) {
+        if (eventId == EventImpl::MOUSEDOWN_EVENT && !me->defaultPrevented()) {
             // Focus should be shifted on mouse down, not on a click.  -dwh
             // Blur current focus node when a link/button is clicked; this
             // is expected by some sites that rely on onChange handlers running
Comment 4 Maksim Orlovich 2009-04-23 17:37:53 UTC 
SVN commit 958236 by orlovich:

Make sure we don't transfer focus on a cancelled mousedown
BUG:190255
BUG:190142


 M  +1 -1      khtmlview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=958236
Comment 5 Maksim Orlovich 2009-04-23 17:39:43 UTC 
SVN commit 958239 by orlovich:

automatically merged revision 958236:
Make sure we don't transfer focus on a cancelled mousedown
BUG:190255
BUG:190142

 M  +1 -1      khtmlview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=958239