← Back to team overview

kicad-developers team mailing list archive

[PATCH] new menu accelerators in the Place menu

 

Hello,
in attachment you can find the simplest patch I can think of to:
1) add the Alt- to all "Place" menu accelerators
2) leave current hotkeys behaviour unchanged

is this whole idea good?

ctrl-alt-L needs to be changed (in the hotkey default set up I think)
because it makes my system log off ! can we changed it? what do you
guys propose? any other hotkey cause conflicts?

I am not sure it will work in windoz/OSX.

Please let me have some feedback

Cheers
Fabrizio
PS Jean-Pierre, I have noticed that you added a close button at the
bottom of the new hotkeys window, this is very cool but as consequence
it does not allow the window to stay open while people use kicad.
Maybe novices, who do not really know yet the hotkeys, wish they could
keep both windows open (as I originally made it), what do you think?
Can we keep the button but allow both window to stay open?
=== modified file 'common/hotkeys_basic.cpp'
--- common/hotkeys_basic.cpp	2011-09-06 07:02:18 +0000
+++ common/hotkeys_basic.cpp	2011-09-06 09:36:00 +0000
@@ -194,7 +194,8 @@
         if( aIsShortCut )
             msg << wxT( "\t" ) << keyname;
         else
-            msg << wxT( " <" ) << keyname << wxT( ">" );
+            //msg << wxT( " <" ) << keyname << wxT( ">" );
+            msg << wxT( "\t" ) << MODIFIER_ALT << keyname;
     }
 
     return msg;
@@ -231,7 +232,8 @@
                 if( aIsShortCut )
                     msg << wxT( "\t" ) << keyname;
                 else
-                    msg << wxT( " <" ) << keyname << wxT( ">" );
+                    //msg << wxT( " <" ) << keyname << wxT( ">" );
+                    msg << wxT( "\t" ) << MODIFIER_ALT << keyname;
 
                 break;
             }


Follow ups