Skip to content
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

Update rect.rst #381

Merged
merged 1 commit into from Feb 6, 2018

Conversation

Projects
None yet
3 participants
@raymonshansen
Copy link
Contributor

raymonshansen commented Jan 25, 2018

Adds: Extended comment regarding behavior of rect.inflate methods.

Spent two hours trying to figure out why my rects behaved strangely when resizing by -1.

Update rect.rst
Adds: Extended comment regarding behavior of rect.inflate methods.
@takluyver

This comment has been minimized.

Copy link
Member

takluyver commented Feb 4, 2018

Thanks - that seems like a useful thing for the docs to note. Could I ask you to have another look at the wording, though? I think it could be clearer.

I see you're from Norway, so English probably isn't your first language - if you're having trouble finding the right words, let me know and I can have a go.

@illume

This comment has been minimized.

Copy link
Member

illume commented Feb 6, 2018

Thanks! 👍 It's nice to document this oddity for others.

The behaviour really does seem like a bug we should fix (one day... lol).

@illume illume merged commit dd87007 into pygame:master Feb 6, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.