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 : [email protected] > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
