Repository navigation
No edges on filled things by default - #5596
Conversation
- for plot, the default marker edge is now the color of the marker - for scatter the marker edge color now follows the face color closes matplotlib#4679
There was a problem hiding this comment.
shouldn't this pull from color instead? That way, there is no visible edge.
There was a problem hiding this comment.
The 'e' here stands for 'errorbar' not 'edge'
|
👍 modulo Ben's concerns about the indentation. I tend to push the continued if lines in an extra 4 spaces. |
b43ca32 to
e8d998f
Compare
|
Updated addressing comments here. |
There was a problem hiding this comment.
histogram steps (for one) go through this path, but sometimes pass fill=False. If that gets ignored, than the patch will have neither fill or stroke in the new scheme here.
|
👍 other than my confusion about a |
|
I think this is good to go. |
There was a problem hiding this comment.
There is a "no newline at end of file" marker here. I've never seen that before. Doesn't PEP8 require a newline at end of file?
There was a problem hiding this comment.
Sorry, missed this comment as I was pressing merge.
There was a problem hiding this comment.
PEP8 doesn't apply as this isn't a Python file ;) But yes, we can go in and add the newline next time we touch this file.
STY: No edges on filled things by default
STY: No edges on filled things by default
|
back ported as 27adae6 |
ghost
mentioned this pull request
Part of the 2.0 style changes.