interface_scrollable: Fix not hold the flag on mouse down 77/113477/2
authorSubodh Kumar <s7158.kumar@samsung.com>
Wed, 19 Aug 2015 15:38:04 +0000 (21:08 +0530)
committerGerrit Code Review <gerrit@review.vlan103.tizen.org>
Wed, 8 Feb 2017 09:54:44 +0000 (01:54 -0800)
when user click fast enough right after scrolling.
becuase there is animators, mouse events aren't invoked at all.

This patch is ported from tizen_2.4.
http://165.213.149.170/gerrit/#/c/75245/

@tizen_fix

Change-Id: I79b8fd201da0dbefe86da0ef43af0ff0c31ab2e8
Signed-off-by: Subodh Kumar <s7158.kumar@samsung.com>
src/lib/elm_interface_scrollable.c

index 787a477..fc2decc 100644 (file)
@@ -2864,10 +2864,16 @@ _elm_scroll_mouse_down_event_cb(void *data,
        (sid->down.momentum_animator) || (sid->scrollto.x.animator) ||
        (sid->scrollto.y.animator))
      {
-        ev->event_flags |= EVAS_EVENT_FLAG_ON_SCROLL |
-          EVAS_EVENT_FLAG_ON_HOLD;
+        //TIZEN_ONLY: (20150819) Do not hold the flag causes click skip
+        //and not required as we are deleting all animators here.
+        //ev->event_flags |= EVAS_EVENT_FLAG_ON_SCROLL |
+        //  EVAS_EVENT_FLAG_ON_HOLD;
+        ev->event_flags |= EVAS_EVENT_FLAG_ON_SCROLL;
+        //
         sid->down.scroll = EINA_TRUE;
-        sid->down.hold = EINA_TRUE;
+        //TIZEN_ONLY: (20150819) Do not hold the flag causes click skip
+        //sid->down.hold = EINA_TRUE;
+        //
         _elm_scroll_anim_stop(sid);
      }
    ELM_SAFE_FREE(sid->scrollto.x.animator, ecore_animator_del);