From: Kim Shinwoo <kimcinoo.efl@gmail.com>
authorKim Shinwoo <kimcinoo.efl@gmail.com>
Mon, 15 Oct 2012 09:14:23 +0000 (09:14 +0000)
committerCarsten Haitzler <raster@rasterman.com>
Mon, 15 Oct 2012 09:14:23 +0000 (09:14 +0000)
commit3998f775f4dbb1a00d3eeb5f043ac5da87771349
tree3f0b33644ad535832b946db4979734aabe132e77
parentc7337c6ae9939e8bca25abc72cd6084eeacaa946
From: Kim Shinwoo <kimcinoo.efl@gmail.com>
Subject: [E-devel] [patch][elementary] ctxpopup - use proper mouse
signal for item

ctxpopup had used its own item not the elm_list. i would like to fix a bug
on selecting item to support user who is using old version ctxpopup.
the reproduce steps are as below.

 0. precondition: ctxpopup displays which has a lot of items, so user
 should scroll to select bottom(or up) side item.
  1. scroll down(or up) ctxpopup item
   2. select one of items

   on step 2,
   the item back ground seems to be clicked. it means part "bg" changes its
state to "clicked" when part "over2" emits "mouse,down,1"
and "default" when part "over2" emits "mouse,up,1".

but, part "over1" have used "ignore_flags: ON_HOLD" with "mouse,up,1", so
on step 2, it cannot emit "elm,action,click"
because elm_scroller uses ELM_EVENT_FLAG_ON_HOLD. so user should
select(click) the item once again.

so in the attached patch, it would be better to remove the "ignore_flags:
ON_HOLD" and use "mouse,clicked,1" instead of "mouse,up,1" as others.
then, please review the attached patch and give feedback. thanks.

SVN revision: 77994
data/themes/widgets/ctxpopup.edc