← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Gerbview: Zoom to selection tool duplication

 

OK.
Please, use patch from first message, that only removes extra tool from
left toolbar.

2017-10-02 17:32 GMT+03:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:

> Nor would I.
>
> On 10/2/2017 3:30 AM, Maciej Sumiński wrote:
> > Hi Konstantin,
> >
> > I would rather not remove the cursor tool from the legacy canvas. Once
> > you select zoom to selection, the only way to revert back to the cursor
> > tool is to press Esc. The rest of the patch seems reasonable to me,
> > though I want to wait for some other comments.
> >
> > Regards,
> > Orson
> >
> > On 10/01/2017 08:32 PM, Константин Барановский wrote:
> >> I noticed that after deleting zoom-to-selection tool from left toolbar,
> >> cursor (no tool) and separator becomes redundant in legacy view mode.
> >> Updated patch in attachment.
> >> It removes zoom-to-selection tool from left toolbar and hides cursor
> with
> >> followed separator in legacy mode. When GAL canvas is active the cursor,
> >> mesurement tool and separator is shows at the top of the toolbar.
> >>
> >> 2017-09-30 22:16 GMT+03:00 Константин Барановский <
> >> baranovskiykonstantin@xxxxxxxxx>:
> >>
> >>> Gerbview has "Zoom to selection" tool present on both toolbars (left
> and
> >>> top).
> >>> Attached patch removes it from left toolbar, so "Zoom to selection"
> stay
> >>> present beside other zooming tools.
> >
> >
> >
> > _______________________________________________
> > 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