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

Bug 476283

Summary: Ensure that users of AbstractFXPinchSpreadGesture can rely on the STARTED, ZOOMED, FINISHED mechanism.
Product: [Tools] GEF Reporter: Matthias Wienand <matthias.wienand>
Component: GEF FXAssignee: Matthias Wienand <matthias.wienand>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 0.1.0   
Target Milestone: 4.0.0 / 3.11.0 (Neon) M4   
Hardware: All   
OS: All   
Whiteboard:

Description Matthias Wienand CLA 2015-09-01 05:20:04 EDT
In some cases, JavaFX will report multiple ZOOM_STARTED events without reporting a ZOOM_FINISHED event in between. However, the ZOOM_FINISHED callbacks need to be invoked, therefore, the callbacks are called when ZOOM_STARTED events are fired consecutively.
Comment 1 Matthias Wienand CLA 2015-09-03 08:52:52 EDT
Actually, multiple ZOOM_STARTED events are no indication of a missing ZOOM_FINISHED event. Moreover, sometimes no ZOOM_FINISHED event is received when the gesture ends (taking the fingers from the touchpad). Therefore, a normal ZOOM event should be reported when multiple ZOOM_STARTED events are received. Additionally a ZOOM_FINISHED event needs to be reported after a small delay where no ZOOM events are received, so that policies can rely on the STARTED, ZOOMED, FINISHED mechanism. I implemented the former, but the latter is still to be done.
Comment 2 Matthias Wienand CLA 2015-09-09 04:31:10 EDT
I changed the title to reflect the actual problem as outlined in my previous comment (i.e. fake events where necessary, e.g. FINISHED after some delay). I removed the target milestone as I am uncertain if we can thoroughly test changes to the pinch/spread gesture for Mars.
Comment 3 Matthias Wienand CLA 2015-11-27 05:39:14 EST
It seems to be fixed by now (cannot be reproduced anymore). Therefore, I resolve this ticket as fixed for 3.11.0M4.