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

feat: allow empty value for /pgf/arrow keys/fill #1352

Closed
wants to merge 0 commits into from

Conversation

hmenke
Copy link
Member

@hmenke hmenke commented Jul 15, 2024

Motivation for this change

Fixes #1348

Checklist

Please signoff your commits to explicitly state your agreement to the Developer Certificate of Origin. If that is not possible you may check the boxes below instead:

Copy link
Member

@muzimuzhi muzimuzhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need a Changelog entry.

tex/generic/pgf/basiclayer/pgfcorearrows.code.tex Outdated Show resolved Hide resolved
Copy link
Member

@muzimuzhi muzimuzhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And a typo on line 830, option -> options:

-    When you use both the |color| and |fill| option, the |color| option must
+    When you use both the |color| and |fill| options, the |color| option must

doc/generic/pgf/pgfmanual-en-tikz-arrows.tex Outdated Show resolved Hide resolved
@hmenke hmenke force-pushed the arrows branch 3 times, most recently from d715236 to ffb5f4f Compare July 19, 2024 08:10
Rmano added a commit to circuitikz/circuitikz that referenced this pull request Jul 19, 2024
Add a new type of arrow to help drawing Jack connectors

Fixes #806 

See also  pgf-tikz/pgf#1352
Thanks to @Skillmon
pgf-tikz/pgf#1348 (comment)
@muzimuzhi
Copy link
Member

@hmenke Do you think this needs a new test?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Adding the opposite of open to arrow tips definitions
2 participants