Skip to content

Support linestyle='none' in Patch#18633

Merged
jklymak merged 2 commits into
matplotlib:masterfrom
QuLogic:fix_linestyle_none
Oct 2, 2020
Merged

Support linestyle='none' in Patch#18633
jklymak merged 2 commits into
matplotlib:masterfrom
QuLogic:fix_linestyle_none

Conversation

@QuLogic

@QuLogic QuLogic commented Oct 2, 2020

Copy link
Copy Markdown
Member

PR Summary

This rebases the branch in #5696 by @Rezangyal, plus a small cleanup, and adds a test.

Fixes #5519.

PR Checklist

  • Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (run flake8 on changed files to check).
  • [n/a] New features are documented, with examples if plot related.
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • Conforms to Matplotlib style conventions (install flake8-docstrings and pydocstyle<4 and run flake8 --docstring-convention=all).
  • [n/a] New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
  • API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).

Rezangyal and others added 2 commits October 2, 2020 03:23
If patch.linestyle  is "none", "None", " ", "" then no edge rendered (as
in Line2D).
@QuLogic QuLogic added the API: consistency Consistency of the matplotlib API, including naming, behavior, defaults, … label Oct 2, 2020
@tacaswell tacaswell added this to the v3.4.0 milestone Oct 2, 2020
@jklymak jklymak merged commit 8699958 into matplotlib:master Oct 2, 2020
@QuLogic QuLogic deleted the fix_linestyle_none branch October 2, 2020 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API: consistency Consistency of the matplotlib API, including naming, behavior, defaults, …

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The linestyle 'None', ' ' and '' not supported by PathPatch.

4 participants