On Fri, May 1, 2009 at 4:02 AM, Sergey Semernin
<sergey.semer...@gmail.com> wrote:
> Hello, All.
>
> This patch adding keyboard shortcuts to efm_nav module.
> When efm_nav placed on efm2 window toolbar, next shortcuts are activated:

Just a small comment, other than that looks good to be included:
> @@ -175,11 +186,11 @@
>      {
>       case E_GADCON_ORIENT_TOP:
>       case E_GADCON_ORIENT_BOTTOM:
> -       e_gadcon_client_aspect_set(gcc, 16 * 3, 16);
> +       e_gadcon_client_aspect_set(gcc, 16 * 4, 16);
>        break;
>       case E_GADCON_ORIENT_LEFT:
>       case E_GADCON_ORIENT_RIGHT:
> -       e_gadcon_client_aspect_set(gcc, 16, 16 * 3);
> +       e_gadcon_client_aspect_set(gcc, 16, 16 * 4);
>        break;
>       default:
>        break;

why is that?


-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: barbi...@gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202

------------------------------------------------------------------------------
Register Now & Save for Velocity, the Web Performance & Operations 
Conference from O'Reilly Media. Velocity features a full day of 
expert-led, hands-on workshops and two days of sessions from industry 
leaders in dedicated Performance & Operations tracks. Use code vel09scf 
and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to