Page MenuHomePhabricator

efl/gesture: correctly handle pointer going off canvas during gestures
Needs ReviewPublic

Authored by zmike on Wed, Jan 8, 11:44 AM.

Details

Reviewers
woohyun
CHAN
Summary

this should immediately cancel an existing gesture and then ignore new
events until the pointer is once again on the canvas

Depends on D11029

Diff Detail

Repository
rEFL core/efl
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 15449
zmike created this revision.Wed, Jan 8, 11:44 AM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

zmike requested review of this revision.Wed, Jan 8, 11:44 AM
zmike added a child revision: D11053: efl/gesture: formatting.
cedric added a subscriber: woohyun.

I am not sure we want this behavior, maybe @woohyun or someone else in Korea would have an opinion on this.

zmike updated this revision to Diff 28130.Mon, Jan 13, 12:12 PM
zmike edited the summary of this revision. (Show Details)

rebase

CHAN requested changes to this revision.Thu, Jan 16, 9:03 PM
CHAN added a subscriber: CHAN.

@zmike Thank you for the commit.

In the previous gesture layer and other platform behaviors, by default, if flick is started inside the target object,
it will track even if the mouse event is outside the target area. :(

Although this patch seems meaningful to me it does not seem to be compatible with the existing behavior learned by the user.

This revision now requires changes to proceed.Thu, Jan 16, 9:03 PM
zmike added a comment.Fri, Jan 17, 6:11 AM
In D11052#210253, @CHAN wrote:

@zmike Thank you for the commit.

In the previous gesture layer and other platform behaviors, by default, if flick is started inside the target object,

it will track even if the mouse event is outside the target area. :(

Although this patch seems meaningful to me it does not seem to be compatible with the existing behavior learned by the user.

This patch attempts to bring parity to running on a desktop vs a touch screen. If you flick a finger off the display of a touch screen, you cannot continue to receive events.

If we don't want this then that's fine as well, and thanks for checking.