kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #32045
Re: [PATCH] Fix drag wire issue
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Tue, 28 Nov 2017 19:39:22 -0500
-
In-reply-to:
<CALHbTmZwiiN00bgvtRa43mnO2av77=eo_OaGmPd-1nZE48BxgQ@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0
Hey Seth,
I just tested this patch on Debian testing against f992d144d90 and I got
the exact same behavior as I did on windows. Is it possible this patch
was created after some of your other wire/junction work. I've attached
a screen capture to show the behavior. You can see the results of the
drag undo as well. HTH.
Cheers,
Wayne
On 11/28/2017 04:44 PM, Seth Hillbrand wrote:
> Hi Wayne-
>
> Thanks for testing. Oddly, I do not observe either of the effects you
> describe on Mac or Linux (Debian), based off master (f992d144d). Are
> you running Windows? If so, I'll try to find a VM to replicate this.
>
> Best-
> Seth
>
> On Tue, Nov 28, 2017 at 12:43 PM, Wayne Stambaugh <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>> wrote:
>
> Seth,
>
> I just tested this. The disambiguation menu is shown but it still drags
> both lines and it breaks the undo as well. Maybe the undo part is fixed
> by your other patch set.
>
> Cheers,
>
> Wayne
>
> On 11/27/2017 5:51 PM, Seth Hillbrand wrote:
> > In Eeschema, if you draw two lines on top of each other that are not
> > the same type, e.g. wire and bus; or bus and graphic line, and then
> > hover over the lines and use the 'drag' hotkey, both lines will
> drag at
> > the same time. The correct behavior should show disambiguation.
> >
> > The attached patch corrects this behavior by distinguishing the line
> > types in SCH_COLLECTOR::IsCorner().
> >
> > Reference: https://bugs.launchpad.net/kicad/+bug/1635984
> <https://bugs.launchpad.net/kicad/+bug/1635984>
> >
> > -Seth
> >
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > More help : https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> >
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> More help : https://help.launchpad.net/ListHelp
> <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:
drag-wire-over-bus-patch-test.webm
Description: video/webm
Follow ups
References