← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Delete whole tracks in GAL

 

Hi John,

This is a clever way of solving the problem, I like it. I have just
committed your patch, well done!

Regards,
Orson

On 02/05/2017 02:23 PM, John Beard wrote:
> Hi,
> 
> These patches implement deletion of whole tracks (actually whole
> connections, it won't span an unrouted gap).
> 
> Fixes: https://bugs.launchpad.net/kicad/+bug/1517213
> 
> This in in two parts:
> 
> * Expand the current connection/track/net selection to act on multiple
> selections, not just the first item in the selection list and refactor
> the core of the traversal code in each case.
> * Register the pre-existing "remove alternate" tool action, add a
> concept of "delete flags" to COMMON_TOOLS, and when triggered, use the
> flag to conditionally select whole tracks to delete.
> 
> The idea of the flags is to allow flexible mappings of modified delete
> actions, in much the same way as modifier keys, but without having to
> commit to a hardcoded key for each flag, so the mappings can be more
> eaisly user defined.
> 
> This patch also reverses the mappings of the delete and backspace
> keys, so that backspace is "normal remove" and "delete" is "remove
> alternate". This keeps the mapping the same between legacy and GAL, as
> per the LP issue. Other than tracks, this won't change anything else,
> as there is as yet no other effect to alt remove, which wasn't used
> until now.
> 
> Cheers,
> 
> John
> 
> 
> 
> _______________________________________________
> 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
> 


References