Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325223 - [navigation] Multi-Hyperlink popup: Command+click deselects instead of executing link
Summary: [navigation] Multi-Hyperlink popup: Command+click deselects instead of execut...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.7   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: 3.7 M2   Edit
Assignee: Markus Keller CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 325230
Blocks:
  Show dependency tree
 
Reported: 2010-09-14 07:01 EDT by Markus Keller CLA
Modified: 2010-09-23 07:58 EDT (History)
1 user (show)

See Also:

Attachments
Workaround (1.09 KB, patch)
2010-09-14 07:51 EDT, Markus Keller CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2010-09-14 07:01:37 EDT 
I20100914-0100 Cocoa, was OK in 3.7 M1

In the Multi-Hyperlink popup, Command+click deselects the item instead of executing the link. Second Command+click works. I'm investigating.
Comment 1 Markus Keller CLA 2010-09-14 07:51:00 EDT 
Created attachment 178808 [details]
Workaround

The problem is bug 325230.

(In reply to bug 325230 comment #1)
> In 3.7 M1, this was not an issue, since the SWT.Selection event didn't get
> through when the table's shell didn't have focus (but MouseDown/MouseUp were
> delivered). Now, the selection event is also delivered in our scenario.

The user experience is quite bad on the Mac, since you have to click twice to activate a link in the popup. Unless bug 325230 gets fixed for M2, I suggest we release this workaround (Mac-only).
Comment 2 Dani Megert CLA 2010-09-14 08:15:49 EDT 
Agree that this is a must fix for M2. Please also check latest M-build.
Comment 3 Markus Keller CLA 2010-09-14 16:29:35 EDT 
3.6.1 is OK (M20100909-0800 Cocoa). Carbon does not have bug 325230 (behaves like win and gtk).
Comment 4 Markus Keller CLA 2010-09-15 04:23:16 EDT 
Fixed in HEAD (without the no-op super.mouseDown(e)).
Comment 5 Markus Keller CLA 2010-09-16 04:24:42 EDT 
Verified in I20100915-2024.
Comment 6 Dani Megert CLA 2010-09-23 07:58:14 EDT 
Te workaround got removed after bug 325230 got fixed.