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 screen.py so we can draw custom images with any pensize or width #69

Closed
wants to merge 2 commits into from
Closed

update screen.py so we can draw custom images with any pensize or width #69

wants to merge 2 commits into from

Conversation

yonghuming
Copy link

i use pensize not width because maybe python beginner have learned python turtle and it use pensize to refer width also in Scratch.
pensize may easier to understand for pupils.

update class SurfacePainter in screen.py ensure user can draw circle with custom pensize
@lordmauve lordmauve added the blocked PRs that cannot be merged for reasons external to this project label Mar 3, 2018
@lordmauve
Copy link
Owner

Due to broken implementation in Pygame <=1.9.3 (#68), this is blocked at least until Pygame 1.9.4 is released.

Repository owner deleted a comment from yonghuming May 18, 2018
@lordmauve
Copy link
Owner

Pygame 1.9.4 is now released.

We could merge this if we ensure we depend on Pygame 1.9.4 or higher.

@SteveClement
Copy link
Contributor

#217 does more or less the same.

If the parameter name 'pensize' is better. Let me know.

@lordmauve
Copy link
Owner

Pygame Zero should align with Pygame on terminology. The width argument in Pygame is a positional-only argument. However, it is documented as width: https://www.pygame.org/docs/ref/draw.html#pygame.draw.circle

@lordmauve
Copy link
Owner

Fixed in d42e3e4

@lordmauve lordmauve closed this Jan 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked PRs that cannot be merged for reasons external to this project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants