X Tutup
Skip to content

BUG: arrowhead drawing code#5392

Merged
QuLogic merged 1 commit intomatplotlib:masterfrom
pizzathief:issue5344
Oct 23, 2016
Merged

BUG: arrowhead drawing code#5392
QuLogic merged 1 commit intomatplotlib:masterfrom
pizzathief:issue5344

Conversation

@pizzathief
Copy link
Copy Markdown
Contributor

removing a left part drawing instruction fixes
an arrowhead drawing problem.

had the instructions for drawing the left part, and
the right part, concatenated the two, without
chopping off enough of the left part instructions.
@pizzathief
Copy link
Copy Markdown
Contributor Author

partly fixes issue #5344, the arrowhead drawing part. arrowhead direction still wrong.

@mdboom
Copy link
Copy Markdown
Member

mdboom commented Nov 4, 2015

Thanks. Have you looked into how hard it will be to fix the arrowhead direction?

@tacaswell
Copy link
Copy Markdown
Member

👍 on merging this as-is

@jenshnielsen
Copy link
Copy Markdown
Member

👍 How easy is it to add a test for this?

@tacaswell tacaswell added this to the 2.0.1 (next bug fix release) milestone Jul 16, 2016
@tacaswell tacaswell self-assigned this Jul 16, 2016
@tacaswell
Copy link
Copy Markdown
Member

Sorry this sat for so long without attention. Retstarting tests against current master.

@QuLogic
Copy link
Copy Markdown
Member

QuLogic commented Oct 23, 2016

Don't see why this can't be merged...

@QuLogic QuLogic merged commit 23afcb4 into matplotlib:master Oct 23, 2016
QuLogic added a commit that referenced this pull request Oct 23, 2016
@QuLogic
Copy link
Copy Markdown
Member

QuLogic commented Oct 23, 2016

Backported to v2.x via 64cee30.

@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
@dstansby dstansby mentioned this pull request Apr 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup