Skip to content

Conversation

@D8H
Copy link
Contributor

@D8H D8H commented Sep 24, 2022

Show how to use the "Ellipse movement" extension to move a player in a snake-like game.

Extension PR:

Show how to use the "Ellipse movement" extension to move a player in a snake-like game.
@D8H D8H added the ✨ New example A new example label Sep 24, 2022
@D8H D8H self-assigned this Sep 24, 2022
@github-actions
Copy link
Contributor

Preview the game(s) changed or added in this Pull Request


This is an automatic message displaying links to the games in this PR - double check the JSON in case of doubt.

@github-actions
Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 2.2%, saving 72 bytes.

Filename Before After Improvement Visual comparison
examples/star-seeker/assets/Red playerShip (1).png 2.65 KB 2.61 KB -1.3% View diff
examples/star-seeker/assets/Silver star.png 637 bytes 601 bytes -5.7% View diff

4396 images did not require optimisation.

@tristanbob tristanbob self-requested a review October 30, 2022 02:25
@tristanbob
Copy link
Contributor

Summary

This is a great example of a one-button game and you did it with only 2 events! Bravo!

Suggestions

  • Use CameraBorderLeft() and similar expressions instead of 0 and SceneWindowWidth. It's more obvious and it works even if the camera moves.
  • Use Object.Width() / 2 instead of 32 so people understand where the number came from (and it still works if they change the object image)

@D8H
Copy link
Contributor Author

D8H commented Nov 3, 2022

  • Use CameraBorderLeft() and similar expressions instead of 0 and SceneWindowWidth. It's more obvious and it works even if the camera moves.
  • Use Object.Width() / 2 instead of 32 so people understand where the number came from (and it still works if they change the object image)

Thanks, I did the changes.

@D8H D8H marked this pull request as ready for review November 3, 2022 23:23
@D8H D8H merged commit b82bc95 into main Nov 6, 2022
@D8H D8H deleted the star-seeker branch November 6, 2022 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ New example A new example

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants