Jerry,

I agree for some of your changes,
but I am not very happy about some others in commit 3544:
Mainly:
-    InitEDA_Appl( wxT( "Eeschema" ), APP_EESCHEMA_T );
+    InitEDA_Appl( wxT( "EESchema" ), APP_EESCHEMA_T );

Eeschema is the official name. If some messages use EESchema, these messages 
should be modified.
No reason to change it.

-    InitEDA_Appl( wxT( "Pcbnew" ), APP_PCBNEW_T );
+    InitEDA_Appl( wxT( "PCBNew" ), APP_PCBNEW_T );

Pcbnew is the official name.
No reason to change it.

-    InitEDA_Appl( wxT( "CvPcb" ), APP_CVPCB_T );
+    InitEDA_Appl( wxT( "CvPCB" ), APP_CVPCB_T );

CvPcb is the official name.
No reason to change it.


But the main issue is the change for zoom centering:
At least is should be discussed, and only activated by an option in preferences 
menu.
The new pan (recently added) was discussed, and activated by an option.

There is no reason to not do the same thing for the new zoom centering

I am expecting your changes soon.

--
Jean-Pierre CHARRAS

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to