Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 513967

Summary: Double clicks on action button to launch the action
Product: [Modeling] Sirius Reporter: Pierre-Charles David <pierre-charles.david>
Component: PropertiesAssignee: Project Inbox <sirius.properties-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: julien.dupont, maxime.porhel, melanie.bats, pierre-charles.david
Version: unspecifiedKeywords: triaged
Target Milestone: 4.1.5   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/93981
https://git.eclipse.org/c/eef/org.eclipse.eef.git/commit/?id=023b0cc57a73a3a665644a9876ab0a4b3fc7ea69
Whiteboard:
Bug Depends on: 501492    
Bug Blocks:    
Attachments:
Description Flags
Use case none

Description Pierre-Charles David CLA 2017-03-21 05:34:38 EDT
+++ This bug was initially created as a clone of Bug #501492 +++

When I want to click on a button in the properties view if this button is not directly visible in the properties view I need to click two time in order the action is launched. 
To reproduce, create a properties view with many labels and after an hyperlink widget with an action button. When you open the properties view pay attenttion that the button is not directly visible, use the scrollbar to see it. Then try to click on the button. The action is not launched you need to click another time to launch it.
It seems that the first click just get the focus and then the second launch the action as expected.
Comment 1 Pierre-Charles David CLA 2017-03-21 05:35:31 EDT
Clone to backport the fix in Sirius 4.1.4. Note that the actual code change is in the EEF runtime, so it will be in EEF 1.9.1.
Comment 2 Eclipse Genie CLA 2017-03-28 09:14:01 EDT
New Gerrit change created: https://git.eclipse.org/r/93981
Comment 4 Pierre-Charles David CLA 2017-04-06 05:27:16 EDT
Fixed (in EEF actually) by 023b0cc57a73a3a665644a9876ab0a4b3fc7ea69.
Comment 5 Julien Dupont CLA 2017-05-18 06:00:43 EDT
Created attachment 268431 [details]
Use case
Comment 6 Pierre-Charles David CLA 2017-06-15 05:24:18 EDT
Available in Sirius 4.1.5, see https://wiki.eclipse.org/Sirius/4.1.5.