Skip to content

Partial fix for #1111: replace some draw_text calls#1240

Merged
einarf merged 1 commit intopythonarcade:developmentfrom
pushfoo:partial_text_object_promotion
Jun 17, 2022
Merged

Partial fix for #1111: replace some draw_text calls#1240
einarf merged 1 commit intopythonarcade:developmentfrom
pushfoo:partial_text_object_promotion

Conversation

@pushfoo
Copy link
Member

@pushfoo pushfoo commented Jun 17, 2022

Built locally and viewed in browser. Includes tweaks to highlighted line numbers.

Let me know if the vertical slice of changes here is acceptable. If it is, I'll keep working on #1111.

Changes made:

  • Sprite Collect Coins: Replace arcade.draw_text and .center_* with arcade.Text and .position
  • Acceleration and Friction: Replace arcade.draw_text and .center_* with arcade.Text and .position
  • Game Controller/Joystick: Replace .center_* in initialization with .position

@pushfoo pushfoo force-pushed the partial_text_object_promotion branch from 624634d to 9242f76 Compare June 17, 2022 19:46
* Sprite Collect Coins: Replace draw_text and .center_* with Text and .position

* Acceleration and Friction: Replace draw_text and .center_* with Text and .position

* Game Controller/Joystick: Replace .center_* in initialization with .position
@pushfoo pushfoo force-pushed the partial_text_object_promotion branch from 9242f76 to 8f18833 Compare June 17, 2022 19:48
@einarf
Copy link
Member

einarf commented Jun 17, 2022

Looks reasonable. A lot of stuff that needs cleanup in examples. I have done a few passes already.

@einarf einarf merged commit b49377b into pythonarcade:development Jun 17, 2022
@pushfoo pushfoo deleted the partial_text_object_promotion branch June 17, 2022 19:55
Sign up for free to 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.

2 participants