← Back to team overview

kicad-developers team mailing list archive

Re: Control state handling changes.

 

Wayne,

I don't know if your work affects the lib editor but I can't add any pin to
a new part.

Tony

On Mon, Feb 21, 2011 at 9:12 PM, Wayne Stambaugh <wstambaugh@xxxxxxxxx>wrote:

> I just committed a large change set to replace all of the current control
> state
> handling code with wxUpdateUIEvent handlers.  It is a much larger change
> set
> than I typically commit but I had to do it this way due to the now defunct
> SetToolbar() code.  I may have broken some enable/disable and/or
> check/uncheck
> behavior in menus and tool bars.  Please let me know if you find any
> control
> state behavior issues so I can fix them.  Thank you for your patience.
>
> Going forward, please use wxUpdateUIEvent to handle all tool bar and menu
> state
> settings.  This is what wxUpdateUIEvent is designed for and it works very
> well.
>  It will also simplify your control state handling code design.  Do not use
> ToggleTool in a wxUpdateUIEvent (doesn't work on GTK).  Use the event
> Check()
> method instead.  Do not call the control Refresh() method in the event
> handler
> or you will create an infinite event loop (I found this out the hard way).
>
> Thanks,
>
> Wayne
>
> _______________________________________________
> 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
>

Follow ups

References