← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Change track width setting to dropbox

 

Looks good to me.  Orson, would you please merge this when you get a chance?

On 8/25/2017 9:51 AM, Maciej Sumiński wrote:
> Hi Matthias,
> 
> Your patch looks good to me. I would love to see the same change done
> for via hole and diameter text controls.

Even better would be to derive an object (EDA_COORDINATE_VALIDATOR?)
from wxTextValidator or one of it's decedents which can be used by any
control that allows text editing.  Rather than use a separate static
text control to display the units, just embed the units in the text
control itself which can be edited by the user including entering
coordinates in units other than the current unit type.  Just food for
thought.

> 
> Regarding inch vs mil, IMHO mils are better suited for PCB design.
> Still, I would rather keep pcbnew consistent, so I suppose we should
> stay with inches for the time being.
> 
> Regards,
> Orson
> 
> On 08/24/2017 09:16 PM, Mathias Grimmberger wrote:
>>
>> Hi all,
>>
>> Mathias Grimmberger <mgri@xxxxxxxxxxxxx> writes:
>>>
>>> Hi Wayne,
>>>
>>> Wayne Stambaugh <stambaughw@xxxxxxxxx> writes:
>>>
>>>> Rather than add a wxComboBox to the WX_UNIT_BINDER object, wouldn't be
>>>> be cleaner to change wxTextCtrl* to wxTextEntry* which wxComboBox and
>>>> wxTextCtrl (along with several others) are derived from?
>>>
>>> Yes, of course, I will change that.
>>
>> Here is the version of the patch with a nicer modification to
>> WX_UNIT_BINDER.
>>
>> BTW, I noticed that displaying e.g. track width in inches feels somehow
>> strange, would people like having selected values displayed in mils?
>>
>>
>> Have fun,
>>
>> MGri
>>
>>
>>
>> _______________________________________________
>> 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