Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite path formation #418

Merged
merged 2 commits into from Jul 1, 2020
Merged

Rewrite path formation #418

merged 2 commits into from Jul 1, 2020

Conversation

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Jul 1, 2020

This basically removes the patches introduced for issue #58 and #76

Reverts these:

Then, it carefully builds the anchors without moving the start of the path, and then join it with a leading --

Fixes #417
Fixes #76
Fixes #58

@Rmano Rmano requested review from der-stefan, mredaelli and sistlind Jul 1, 2020
@Rmano Rmano self-assigned this Jul 1, 2020
@Rmano
Copy link
Collaborator Author

@Rmano Rmano commented Jul 1, 2020

For cleanliness I reopened #76

Fixes #76 again

@Rmano
Copy link
Collaborator Author

@Rmano Rmano commented Jul 1, 2020

I think I will merge this if there are no comments. Seems correct, and compiles the manual just fine. Moreover, if I don't merge and release nobody will test it... ;-)

Rmano and others added 2 commits Jul 1, 2020
This basically remove the patches introduced for issue circuitikz#58 and circuitikz#76

Reverts these:
   circuitikz@6efc2ee
   circuitikz@352d2bf

Then, it carefully builds the anchors without moving the start of the
path, and then join it with a leading --.
Notice that it worked because of (basically) a circuitikz bug. The path
was drawn up and down several times, so that the default pos=0.5 was
putting the labels to the end...
@Rmano Rmano force-pushed the rewrite-path-formation branch from 417ee41 to 7072571 Jul 1, 2020
@Rmano Rmano merged commit 67851f3 into circuitikz:master Jul 1, 2020
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

1 participant