Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353720 - SWTBotTreeItem.doubleClick() not setting mouse button
Summary: SWTBotTreeItem.doubleClick() not setting mouse button
Status: RESOLVED FIXED
Alias: None
Product: SWTBot
Classification: Technology
Component: SWTBot (show other bugs)
Version: 2.0.4   Edit
Hardware: PC Linux
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-03 05:28 EDT by Lorenzo Bettini CLA
Modified: 2011-09-21 09:52 EDT (History)
1 user (show)

See Also:

Attachments
mylyn/context/zip (3.41 KB, application/octet-stream)
2011-08-03 05:32 EDT, Lorenzo Bettini CLA 		no flags Details
patch for fixing this bug (2.69 KB, patch)
2011-08-03 05:43 EDT, Lorenzo Bettini CLA 		mariot.chauvin: iplog+
Details | Diff
mylyn/context/zip (3.65 KB, application/octet-stream)
2011-08-03 05:43 EDT, Lorenzo Bettini CLA 		no flags Details
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 Lorenzo Bettini CLA 2011-08-03 05:28:16 EDT 
I am using SWTBotMenuItem to post some doubleClick events to a Tree.

The events got posted correctly but without the mouse button which makes impossible to test mouse listeners which check the button pressed.
Comment 1 Lorenzo Bettini CLA 2011-08-03 05:32:38 EDT 
Created attachment 200785 [details]
mylyn/context/zip

method doubleClick and the corresponding test
Comment 2 Lorenzo Bettini CLA 2011-08-03 05:43:52 EDT 
Created attachment 200788 [details]
patch for fixing this bug
Comment 3 Lorenzo Bettini CLA 2011-08-03 05:43:53 EDT 
Created attachment 200789 [details]
mylyn/context/zip
Comment 4 Lorenzo Bettini CLA 2011-08-03 05:44:13 EDT 
This is similar to bug 308619 and I borrowed from that bug for the patch attached (there is also a patch for the unit test).
Comment 5 Mariot Chauvin CLA 2011-09-21 08:30:57 EDT 
fixed with commit f0ced0fb1faad6fbaa4c606bc3884d37fe863ed6
Comment 6 Mariot Chauvin CLA 2011-09-21 08:31:41 EDT 
(In reply to comment #5)
> fixed with commit f0ced0fb1faad6fbaa4c606bc3884d37fe863ed6

Thanks for the patch !
Comment 7 Lorenzo Bettini CLA 2011-09-21 09:15:16 EDT 
but I don't see the commit... have you pushed?
Comment 8 Mariot Chauvin CLA 2011-09-21 09:52:16 EDT 
(In reply to comment #7)
> but I don't see the commit... have you pushed?

yep I pushed, see http://git.eclipse.org/c/swtbot/org.eclipse.swtbot.git/commit/?id=f0ced0fb1faad6fbaa4c606bc3884d37fe863ed6