kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #31928
Re: [PATCH] Eeschema automatic manage junctions
-
To:
<kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Sumiński <maciej.suminski@xxxxxxx>
-
Date:
Fri, 24 Nov 2017 15:50:18 +0100
-
Authentication-results:
spf=pass (sender IP is 188.184.36.48) smtp.mailfrom=cern.ch;
-
In-reply-to:
<CALHbTmaPA=sEDSuOXMjWOUb_NP67guDSZdWfmZy8-GW-HZDMOw@mail.gmail.com>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0
Hi Seth,
I tested the branch and I consider it a significant improvement to the
way junctions are handled. I confirm it fixes the four bugs mentioned in
patch 0011, apart from a single case when a parallel component is
dragged out. In this case junctions are not still not auto deleted (see
[1], the rightmost picture "Junction is not auto deleted"), but at least
the overlapping wires are merged. It is not a big deal IMHO, especially
the patch set fixes a lot of other issues.
It has not been stated explicitly, but I thought we will use
NEW/REMOVE/CHANGE tags for listing changes that are perceived by the
user. Therefore:
CHANGE: DeleteItem removes junctions that are no longer needed.
informs the user about an improvement, but:
CHANGE: DeleteItemsInList shares the code for DeleteItem.
seems to be too detailed and does not need to be reflected in the user
documentation.
What do you think? I will try to come up with a short paragraph that can
be added to the website to make things clear.
Regards,
Orson
1.
https://launchpadlibrarian.net/213748651/KICAD%20-%20BUG%20REPORT%20-%20EESCHEMA%20-%20SOME%20DRAG%20ISSUES.jpg
On 11/22/2017 09:07 PM, Seth Hillbrand wrote:
> Updated patchset for this proposal, rebased to master. I've also updated
> the commit messages to match the CHANGE:/NEW: format and added one new bug
> from launchpad that this addresses.
>
> -Seth
>
> On Wed, Nov 8, 2017 at 3:59 AM, Nick Østergaard <oe.nick@xxxxxxxxx> wrote:
>
>> For that specific issue with the junction drawing, there is a patch in the
>> thread "[Kicad-developers] [PATCH] Draw junctions last"
>>
>> 2017-11-03 13:12 GMT+01:00 Jon Evans <jon@xxxxxxxxxxxxx>:
>>
>>> I looked at fixing this and some other related things, and decided to
>>> just wait for the GAL port. There will need to be huge refactoring of the
>>> eeschema draw code as part of that effort, so putting much effort into
>>> making the wxDC drawing better seems not worth it.
>>>
>>> -Jon
>>>
>>> On Nov 3, 2017 00:08, "Kevin Cozens" <kevin@xxxxxxxxx> wrote:
>>>
>>> On 2017-11-02 06:31 PM, Seth Hillbrand wrote:
>>>
>>>> Please let me know if there are any additional issues or suggestions for
>>>> improvement.
>>>>
>>>
>>> How difficult would it be to have junctions draw last on schematics?
>>> There is a minor negative visual effect when you have a component with one
>>> end joined to a wire by a junction and you replace the component.
>>>
>>> When you replace the component the pin of the component is now seen
>>> extending through the round disc of the junction to the center of the
>>> junction. I prefer to always see just the full round disc of a junction
>>> mark even if I have replaced a component since placing the junction.
>>>
>>> --
>>> Cheers!
>>>
>>> Kevin.
>>>
>>> http://www.ve3syb.ca/ |"Nerds make the shiny things that
>>> distract
>>> Owner of Elecraft K2 #2172 | the mouth-breathers, and that's why
>>> we're
>>> | powerful!"
>>> #include <disclaimer/favourite> | --Chris Hardwick
>>>
>>>
>>> _______________________________________________
>>> 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
>>>
>>>
>>
>> _______________________________________________
>> 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
>
Attachment:
signature.asc
Description: OpenPGP digital signature
Follow ups
References
-
[PATCH] Eeschema automatic manage junctions
From: Seth Hillbrand, 2017-10-17
-
Re: [PATCH] Eeschema automatic manage junctions
From: Seth Hillbrand, 2017-10-28
-
Re: [PATCH] Eeschema automatic manage junctions
From: Nick Østergaard, 2017-10-28
-
Re: [PATCH] Eeschema automatic manage junctions
From: Seth Hillbrand, 2017-10-28
-
Re: [PATCH] Eeschema automatic manage junctions
From: Nick Østergaard, 2017-10-28
-
Re: [PATCH] Eeschema automatic manage junctions
From: Seth Hillbrand, 2017-10-29
-
Re: [PATCH] Eeschema automatic manage junctions
From: Seth Hillbrand, 2017-11-02
-
Re: [PATCH] Eeschema automatic manage junctions
From: Kevin Cozens, 2017-11-03
-
Re: [PATCH] Eeschema automatic manage junctions
From: Jon Evans, 2017-11-03
-
Re: [PATCH] Eeschema automatic manage junctions
From: Nick Østergaard, 2017-11-08
-
Re: [PATCH] Eeschema automatic manage junctions
From: Seth Hillbrand, 2017-11-22