FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Changed offset reference, add small doc by story645 · Pull Request #23223 · matplotlib/matplotlib · GitHub

Repository navigation

Changed offset reference, add small doc - #23223

Merged
timhoffm merged 1 commit into
matplotlib:mainfrom
story645:story645-offset
Jun 8, 2022
Merged

timhoffm merged 1 commit into
matplotlib:mainfrom
story645:story645-offset

Conversation

story645 commented Jun 8, 2022

Copy link
Copy Markdown
Member

PR Summary

As mentioned in #23208, the docs currently link to a post script reference (the blue book) that is on a system with restricted access, so I changed the link to the Red book since I found that on an official adobe site. Also changed the page # to where the reference is in the red book & took a first stab at copy pasting in the official description for offset.

I think this partially addresses #23220

story645 force-pushed the story645-offset branch 3 times, most recently from ab5ef05 to ea46826 Compare June 8, 2022 15:18

oscargus 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks good! There are also a bunch of set_linestyle functions (three), but since I touch those in #23056 it is probably better that I try to incorporate something similar there.

timhoffm added this to the v3.6.0 milestone Jun 8, 2022
timhoffm merged commit 055452f into matplotlib:main Jun 8, 2022
story645 deleted the story645-offset branch June 8, 2022 17:51
rcomer mentioned this pull request Jun 12, 2022
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL