kicad-developers team mailing list archive
Mailing list archive
Re: [Patch] Unify close menu item creation for non-main windows
I merged your patch. Thanks for your contribution!
> On 14 Aug 2019, at 00:40, Ian McInerney <Ian.S.McInerney@xxxxxxxx> wrote:
> Right now the addition of the Close item for the non-main windows is scattered in the menu creation functions. This means it is easy to leave out the accelerator key entry when adding a new one (yes, I forgot that in cvpcb...). This patch will push the creation of the menu item into a common method in CONDITIONAL_MENU that is then just called by each window. It also introduces a tooltip for the item that includes the app name. This unifies all the close items, and ensures they have the proper CTRL-W accelerator key assigned.
> Note that this does not unify the one for the simulation window. That window is a wxFormbuilder window so its menu is not a CONDITIONAL_MENU and can't use this.
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp