Hello, belated review :)

1. Use Evas_Smart_Cb_Description for smart callbacks. I fixed it in SVN.
http://trac.enlightenment.org/e/changeset/60014

2. Changed signal name.
I changed reorder signal name from "elm,state,reorder_enabled" to
"elm,state,reorder,enabled".
This is more EFL-like I guess.
I changed genlist's as well.

Thanks.
Daniel Juyung Seo (SeoZ)


On Fri, Jun 3, 2011 at 4:13 PM, Carsten Haitzler <ras...@rasterman.com> wrote:
> On Fri, 03 Jun 2011 16:04:51 +0900 Jeonghyun Yun <jh0506....@samsung.com> 
> said:
>
> now THATS good stuff! you can see when it goes into reorder mode... yay! :)
> nice! in svn!
>
>> Hello,
>>
>> I added theme stuff to handle the reorder mode.
>> Please check this patch again.
>>
>> Thanks, raster.
>>
>>
>> -----Original Message-----
>> From: Carsten Haitzler (The Rasterman) [mailto:ras...@rasterman.com]
>> Sent: Thursday, June 02, 2011 4:10 PM
>> To: Jeonghyun Yun
>> Cc: enlightenment-devel@lists.sourceforge.net
>> Subject: Re: [E-devel] [Patch] elm_gengrid - added new feature : gengrid
>> reorder mode
>>
>> On Mon, 30 May 2011 19:41:24 +0900 Jeonghyun Yun <jh0506....@samsung.com>
>> said:
>>
>> this seems good. no theme stuff to handle the reorder mode change on the
>> item
>> tho.. shame :(
>>
>> > Dear All.
>> >
>> > I added a new feature "reorder mode" of gengrid refer to genlist reorder
>> > mode.
>> >
>> > After set this reorder API, if user press an item (long press), the item
>> can
>> > be moved another position.
>> > Then if the user releases the item, the item will be reordered in gengrid.
>> >
>> > Please review this patch.
>> >
>> > Thanks.
>> >
>> > [API]
>> > ==================================================================
>> > /**
>> >  * Set the reorder mode
>> >  *
>> >  * @param obj The Gengrid object
>> >  * @param reorder_mode The reorder mode
>> >  * (EINA_TRUE = on, EINA_FALSE = off)
>> >  *
>> >  * @ingroup Gengrid
>> >  */
>> > EAPI void
>> > elm_gengrid_reorder_mode_set(Evas_Object *obj, Eina_Bool reorder_mode)
>> >
>> > /**
>> >  * Get the reorder mode
>> >  *
>> >  * @param obj The Gengrid object
>> >  * @return The reorder mode
>> >  * (EINA_TRUE = on, EINA_FALSE = off)
>> >  *
>> >  * @ingroup Gengrid
>> >  */
>> > EAPI Eina_Bool
>> > elm_gengrid_reorder_mode_get(const Evas_Object *obj)
>> > ==================================================================
>>
>>
>> --
>> ------------- Codito, ergo sum - "I code, therefore I am" --------------
>> The Rasterman (Carsten Haitzler)    ras...@rasterman.com
>
>
> --
> ------------- Codito, ergo sum - "I code, therefore I am" --------------
> The Rasterman (Carsten Haitzler)    ras...@rasterman.com
>
>
> ------------------------------------------------------------------------------
> Simplify data backup and recovery for your virtual environment with vRanger.
> Installation's a snap, and flexible recovery options mean your data is safe,
> secure and there when you need it. Discover what all the cheering's about.
> Get your free trial download today.
> http://p.sf.net/sfu/quest-dev2dev2
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to