← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] GerbView GAL support

 

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
> 


Attachment: signature.asc
Description: OpenPGP digital signature


Follow ups

References