Skip to content

DOC: Fixes the links in the see-also section of Axes.get_tightbbox - #14764

Merged
jklymak merged 1 commit into
matplotlib:masterfrom
alexrudy:fix-broken-see-also-links
Jul 15, 2019
Merged

jklymak merged 1 commit into
matplotlib:masterfrom
alexrudy:fix-broken-see-also-links

Conversation

@alexrudy

Copy link
Copy Markdown
Contributor

PR Summary

Two of the members in the "See Also" section of Axes.get_tightbbox referred to other python objects which don't exist. I have inferred the correct values, but I could be wrong!

@alexrudy alexrudy changed the title Fixes the links in the see-also section of Axes.get_tightbbox DOC: Fixes the links in the see-also section of Axes.get_tightbbox Jul 13, 2019

@bsipocz bsipocz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@bsipocz

bsipocz commented Jul 14, 2019

Copy link
Copy Markdown
Contributor

Btw, I can wholeheartedly recommend @Cadair's https://github.com/apps/giles bot to provide a single link in the checks section above to the rendered html docs.

@jklymak jklymak left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ooops, my bad!

@jklymak
jklymak merged commit f9606c1 into matplotlib:master Jul 15, 2019
@QuLogic QuLogic added this to the v3.1.2 milestone Jul 17, 2019
@QuLogic

QuLogic commented Jul 17, 2019

Copy link
Copy Markdown
Member

@meeseeksdev backport to v3.1.x

meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jul 17, 2019
QuLogic added a commit that referenced this pull request Jul 18, 2019
…764-on-v3.1.x

Backport PR #14764 on branch v3.1.x (DOC: Fixes the links in the see-also section of Axes.get_tightbbox)
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.

5 participants