Overall idea looks good.

1) wrap 80 chars
> evas_object_event_callback_add(wd->rect, EVAS_CALLBACK_RESIZE, 
> _rect_resize_cb, obj);

This is too long. EFL sources are written based on 80 chars per line policy.
Please wrap them correctly.

2) remove unrelated changes
Check config/profile.src, src/bin/test_map.c

3) setter/getter apis
We should also set the configuration by apis.
ex) elm_password_show_last_get/set
It looks like glayer has no configurable apis. I need Tom's comment for this.

Thanks.

Daniel Juyung Seo (SeoZ)

On Thu, Dec 15, 2011 at 2:14 PM, Bluezery <ohpo...@gmail.com> wrote:
> Dear all,
>
> Elementary can be used for all because EFL is an open source.
> But multi Touch event can have paten issue such as pinch zooming action.
> So, I add "finger(pinch) zooming action" to elm_config to can be
> enabled or disabled.
> This configuration value can be used for diverse product.
> It's my pleasure if Mr. Hacohen have time to review this. :-)
>
> Only gesture layer recognizes this value if all other elementary is
> based on gesture layer.
> But elm map implemented pinch zooming by itself. I will change elm_map
> to use gesture layer soon.
> For the consistency, elm_map read this value temporarily. It will
> removed later after gesture layer is applied.
>
> Please review this patch.
>
> --
> BRs,
> Kim.
>
> ------------------------------------------------------------------------------
> 10 Tips for Better Server Consolidation
> Server virtualization is being driven by many needs.
> But none more important than the need to reduce IT complexity
> while improving strategic productivity.  Learn More!
> http://www.accelacomm.com/jaw/sdnl/114/51507609/
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>

------------------------------------------------------------------------------
10 Tips for Better Server Consolidation
Server virtualization is being driven by many needs.  
But none more important than the need to reduce IT complexity 
while improving strategic productivity.  Learn More! 
http://www.accelacomm.com/jaw/sdnl/114/51507609/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to