yeap, it seems nice.
btw, i have a question.

-   if (eina_list_count(wd->items) < 1)
+   list = elm_object_item_widget_get(ctxpopup_it->list_item);
+
+   if (eina_list_count(elm_list_items_get(list)) < 2)
      {
+        elm_object_item_del(ctxpopup_it->list_item);
         evas_object_hide(WIDGET(ctxpopup_it));
         return EINA_TRUE;
      }

is there any special reason to check '2' case?

On Wed, Mar 21, 2012 at 6:19 PM, cnook <kimci...@gmail.com> wrote:
> Dear All, Hello.
>
> The patch is for using elm_list as a item controller of elc_ctxpopup.
> I have learned that there was an opinion to change the internal structure
> of ctxpopup using the list.
> It would be able to match up to your expectation. but it is just a draft
> version of using list in ctxpopup.
> So, It would be pretty good to me if you give any feedback on this. Thanks.
>
> Sincerely,
> Shinwoo Kim.
>
> ------------------------------------------------------------------------------
> This SF email is sponsosred by:
> Try Windows Azure free for 90 days Click Here
> http://p.sf.net/sfu/sfd2d-msazure
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>

------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to