← Back to team overview

kicad-developers team mailing list archive

Re: Modedit graphical item properties dialog patch

 

On 1/26/2017 2:52 AM, Maciej Sumiński wrote:
> On 01/25/2017 08:14 PM, Diogo Condeço wrote:
>> Hi all,
>>
>> This patch corrects the CMD+C cancel dialog bug on macos... There are still
>> some both in the footprint editor, and the board editor. I will try to
>> patch them once i can get some free time.
> 
> I wonder if FixOSXCancelButtonIssue() call could be moved to OnShow
> event handler in DIALOG_SHIM. This way it would be invoked automatically
> for every dialog inheriting from DIALOG_SHIM (and I guess this covers
> majority of cases) without manually fixing them one by one.
> 
>> There is still the setfocus issue, which I will try to get later on... But
>> I would like to go through the code, since I don't really know if there is
>> the need to have two different classes for graphical_item_properties... One
>> for the FP editor another for the Board Editor... << Any input on this?
> 
> There is a lot of repeated code, but also a few subtle differences. If
> everything could be covered by the same code, I am for it.

I was thinking the same thing as each of these patches has very similar
code.  This is why DIALOG_SHIM exists to provide features and fixes to
wxDialog.  If they osx focus fix can be handled in DIALOG_SHIM, than I
would prefer it be fixed there rather than every single dialog source file.

> 
> Regards,
> Orson
> 
>> Thanks,
>>
>>
>>
>> _______________________________________________
>> 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
>>
> 
> 
> 
> 
> _______________________________________________
> 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