← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] GerbView GAL support

 

Hi Orson,

Give this a shot in your branch.  It should work in pcbnew also now.

-Jon

On Wed, Sep 20, 2017 at 9:28 AM, Jon Evans <jon@xxxxxxxxxxxxx> wrote:

> Hi Orson,
>
> Thank you for staging this for merge on your branch.  I checked and you do
> have all the patches.
>
> 1) Yes I planned on refactoring the selection tool once things stabilized
> with the highlighting etc.
> 2) Do you mean when you are highlighting Gerber X2 attributes, or when you
> are deselecting things, or something else?
> 3) That's a good idea on VIEW_GROUP, I will give it a try and send a patch.
>
> Thanks,
> Jon
>
> On Wed, Sep 20, 2017 at 5:46 AM, Maciej Sumiński <maciej.suminski@xxxxxxx>
> wrote:
>
>> Hi Jon,
>>
>> GALifying GerbView is a huge task, so thank you very much for your work!
>> I have just tested your changes and in my opinion it is in a state that
>> deserves merging and further tests. The new way of item highlighting is
>> awesome, we need to port it to pcbnew as well.
>>
>> For now I keep your patches in a separate branch, with some minor
>> modifications on top of it [1]. Please verify it contains all the needed
>> patches. If nobody objects, I would like to merge it this week.
>>
>> Just a few minor remarks:
>> - It seems there is some code that could be refactored to share it with
>> pcbnew (e.g. selection tool).
>> - 'Clear highlight' operation takes long time to finish (seems more than
>> with the legacy canvas), but I cannot really see what is happening
>> there. If it cannot be easily fixed, perhaps it could set the mouse
>> cursor to busy.
>> - For the new highlighting method: perhaps a more universal way is to
>> create a temporary VIEW_GROUP object containing the selection candidate.
>> This way it can be temporarily displayed on the overlay layer, without
>> modifying the original ViewGetLayer() methods.
>>
>> Regards,
>> Orson
>>
>> 1. https://code.launchpad.net/~orsonmmz/kicad/+git/kicad/+ref/
>> gerbview_gal
>>
>> On 09/18/2017 12:47 AM, Jon Evans wrote:
>> > Hi all,
>> >
>> > The day has finally come!  I have distilled my GerbView GAL branch into
>> a
>> > patchset attached to this email.  Hopefully with this merged into
>> master we
>> > can identify any remaining bugs and clean it up for 5.0.
>> >
>> > Note that this set is split into 5 patches to make review easier, but
>> they
>> > are not intended to compile and work independently.
>> >
>> > Best,
>> > Jon
>> >
>> >
>> >
>> > _______________________________________________
>> > 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
>>
>>
>

Attachment: 0001-Use-VIEW_GROUP-for-candidate-highlighting-enable-in-.patch
Description: Binary data


Follow ups

References