← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] eeschema: add Select button for footprints in single component field edits

 

I forgot about this patch, but I have now applied it and do not see
any issue with it. I cannot comment on the code.

But while testing it out I noticed that above the textbox it just says
"Text" in a label. Shouldn't this be "Footprint name" or something
similar?

Nick

2014-10-14 17:24 GMT+02:00 Wayne Stambaugh <stambaughw@xxxxxxxxxxx>:
> I've already applied the patch to my branch and did some testing.  It
> works as advertized but I just want to make sure we are for the most
> part on board with the changes.  I'll wait a while to make sure there
> are no strong objections before I commit the changes.  Thanks for the
> feedback.
>
> Cheers,
>
> Wayne
>
> On 10/14/2014 11:15 AM, Brian Sidebotham wrote:
>> Sorry, I didn't get much chance to look at it - but I liked the idea
>> of it - I think this is one of the very reasons behind the KiWay work.
>> It's certainly a bonus of that work which we should be reaping.
>>
>> I didn't get a chance to apply and test though. Let me know if you need that.
>>
>> Best Regards,
>>
>> Brian.
>>
>> On 14 October 2014 14:58, Wayne Stambaugh <stambaughw@xxxxxxxxxxx> wrote:
>>> Has anyone else looked at this patch?  If you haven't, it adds a select
>>> button that launches the footprint viewer to select a footprint to the
>>> edit field properties dialog when the footprint field is selected.  I
>>> personally like it but since it's behavioral change I would like some
>>> input before I commit it.
>>>
>>> Thanks,
>>>
>>> Wayne
>>>
>>> On 10/11/2014 1:43 AM, Mark Roszko wrote:
>>>> Hi,
>>>>
>>>> This patch makes the changes to add a selection button to the simple
>>>> dialog that appears when you right-click on a component and choose
>>>> Edit Component > Footprint. The button will open the footprint
>>>> browser. This is just like the component property editor having the
>>>> assign footprint button.
>>>> + Doxygenization of the class.
>>>>
>>>> This also works in the library editor when you click on the footprint
>>>> text and go to edit it.
>>>>
>>>> Patches attached and a bazr branch submitted for merge.
>>>>
>>>> https://code.launchpad.net/~mark-roszko/kicad/eeschema_footprint_pick
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> 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
>>
>
>
> _______________________________________________
> 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