8000 DOC: Add documentation to Text.set_fontfamily by bharatr21 · Pull Request #13949 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

DOC: Add documentation to Text.set_fontfamily #13949

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 3 commits into from
Apr 14, 2019

Conversation

bharatr21
Copy link
Contributor

PR Summary

PR Checklist

  • Has Pytest style unit tests (Not needed; doc change)
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there) (Not a major new feature)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way (Not backward incompatible)

Fix #13880 by adding documentation to Text.set_fontfamily()

Copy link
Member
@timhoffm timhoffm left a comment

Choose a reason for hiding this comment

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

Thanks, only some formatting issues remaining.

@@ -977,6 +977,11 @@ def set_fontfamily(self, fontname):
name or a generic font class name. If the latter, the specific font
names will be looked up in the corresponding rcParams.

If a `Text` instance is constructed with `fontfamily=None`, then the
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
If a `Text` instance is constructed with `fontfamily=None`, then the
If a `Text` instance is constructed with ``fontfamily=None``, then the

@timhoffm timhoffm merged commit c2bfa00 into matplotlib:master Apr 14, 2019
@timhoffm
Copy link
Member

Thanks, and congratulations on your first contribution to Matplotlib! Maybe see you back sometime.

@timhoffm timhoffm added this to the v3.1.0 milestone Apr 14, 2019
@timhoffm
Copy link
Member

@meeseeksdev backport to v3.1.0

@lumberbot-app
Copy link
lumberbot-app bot commented Apr 14, 2019

Something went wrong ... Please have a look at my logs.

It seem that the branch you are trying to backport to does not exists.

@timhoffm
Copy link
Member

@meeseeksdev backport to v3.1.x

@bharatr21
Copy link
Contributor Author
bharatr21 commented Apr 14, 2019 via email

timhoffm added a commit that referenced this pull request Apr 14, 2019
…949-on-v3.1.x

Backport PR #13949 on branch v3.1.x (DOC: Add documentation to Text.set_fontfamily)
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.

Adding documentation for Text.fontfamily default, set_fontfamily(None)?
3 participants
0