Elementary genlist: Fixed bug in single selection mode to prevent
authorseoz <seoz@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Wed, 25 May 2011 03:01:51 +0000 (03:01 +0000)
committerseoz <seoz@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Wed, 25 May 2011 03:01:51 +0000 (03:01 +0000)
multiple selection. Patch by Hyoyoung Chang <hyoyoung.chang@samsung.com>

On Tue, May 24, 2011 at 7:43 PM, Hyoyoung Chang
<hyoyoung.chang@samsung.com> wrote:
Dear Elementary developers.

I made a oneline patch.
It's about in single selection mode of elm_genlist.
In single selection mode, some multi touching can make multi selected items.

It can't easy reproduce at PC environment(like as mouse+keyboard), but if you have multi touching device you can easily reproduce.

Reproduce step is putting two pointing device on genlist.
1.      long pressing one pointer.
2.      Short press another one pointer.
3.      And repeat alternating press two device
Then you can find multi selected items in single selection mode.

So my approach is if a item is not selection and just only
highlighted, then make it with _item_unselection()

git-svn-id: https://svn.enlightenment.org/svn/e/trunk/elementary@59659 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33

src/lib/elm_genlist.c

index 8c70f92..96aef6d 100644 (file)
@@ -1379,6 +1379,7 @@ _mouse_up(void        *data,
    it->wd->mouse_down = EINA_FALSE;
    if (it->wd->multitouched)
      {
+        if ((!it->wd->multi) && (!it->selected) && (it->highlighted)) _item_unhighlight(it);
         if (it->wd->multi_down) return;
         _multi_touch_gesture_eval(data);
         return;