Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Convert named_colors example to use Rectangle #16467

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

Merged
merged 2 commits into from
Feb 11, 2020
Merged

Convert named_colors example to use Rectangle #16467

merged 2 commits into from
Feb 11, 2020

Conversation

asongtoruin
Copy link
Contributor

Converts the named colours example to use Rectangle rather than hlines for ease of future updating. Discussions still ongoing in #15614 as to whether the colour swatches should be given borders or not.

In line with #15614, this changes the named_colors example in the docs
to use a Rectangle rather than drawing lines. This should make it more
flexible to update in future.
@timhoffm timhoffm added this to the v3.3.0 milestone Feb 11, 2020
@timhoffm timhoffm merged commit c91d7ef into matplotlib:master Feb 11, 2020
@story645
Copy link
Member

Is the plan to open a new PR for the borders?

@timhoffm
Copy link
Member

timhoffm commented Feb 11, 2020

Thanks, and congratulations on your first contribution to matplotlib! We'd love to see you back in the future.

Note: I've squashed your commits before merging. We try to keep commits to logical units to keep the history more readable. You can use git commit -a --amend --no-edit and then git push origin [mybranch] --force to update a PR without adding additional commits.

@asongtoruin
Copy link
Contributor Author

@timhoffm apologies - I just accepted through the GH interface, I'll push it through manually next time.

@story645 there didn't seem to be a consensus as to whether we should change it or not, but it's a fairly trivial change I can make for a new PR if you feel like further discussion would be helpful?

@story645
Copy link
Member

Yes, if you don't mind please make a new PR for the border.

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

Successfully merging this pull request may close these issues.

4 participants