Skip to content

Improve NinePatchTexture doc - #1682

Merged
eruvanos merged 8 commits into
pythonarcade:developmentfrom
pushfoo:improve_9patch_doc
Apr 6, 2023
Merged

Improve NinePatchTexture doc#1682
eruvanos merged 8 commits into
pythonarcade:developmentfrom
pushfoo:improve_9patch_doc

Conversation

@pushfoo

@pushfoopushfoo commented Apr 5, 2023

Copy link
Copy Markdown
Member

tl;dr

  1. Clearer phrasing for all NinePatchTexture docstrings
  2. Better behavior diagram, now with arrows & parameter labels!
  3. Stricter flake8 & docstring PEP compliance

While responding to #1675, I looked noticed NinePatchTexture had unclear docstrings. This PR addresses this and serves as a stepping stone to further improvements beyond those already in #1680.

Doc built and tested locally.

Comment threadarcade/gui/nine_patch.py Outdated
@pushfoopushfoo changed the title Improve NinePatchTexture docstringsImprove NinePatchTexture docApr 5, 2023
@pushfoo
pushfoo requested a review from einarfApril 5, 2023 21:10
@eruvanos
eruvanos merged commit 664ae2b into pythonarcade:developmentApr 6, 2023
@pushfoo
pushfoo deleted the improve_9patch_doc branch April 6, 2023 08:23
@pushfoo
pushfoo restored the improve_9patch_doc branch April 11, 2023 18:37
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@pushfoo@einarf@eruvanos