Skip to content

Port game-01 to current plugin snapshot#13

Closed
wjt wants to merge 1 commit intogodotconfrom
push-mzluwosknuvt
Closed

Port game-01 to current plugin snapshot#13
wjt wants to merge 1 commit intogodotconfrom
push-mzluwosknuvt

Conversation

@wjt
Copy link
Copy Markdown
Member

@wjt wjt commented Oct 16, 2024

To get this working I rebuilt the 3 block code programs with exactly the same blocks as before.

I then added a CollisionPolygon2D to the SimpleCharacter. This is because in the old plugin version a 100×100 square CollisionShape2D was unconditionally added; whereas in the new version this is only added if the SimpleCharacter's texture property is set. But the texture property is not set.

We could instead set the texture and drop the Sprite2D2 child. However it's not as trivial as setting the texture property because the sprite is the wrong size and is scaled down to 0.321 its full size by the Sprite2D2.

So, add a CollisionPolygon2D, and adjust the position of the dead zone accordingly.

To get this working I rebuilt the 3 block code programs with exactly the same
blocks as before.

I then added a CollisionPolygon2D to the SimpleCharacter. This is because in
the old plugin version a 100×100 square CollisionShape2D was unconditionally
added; whereas in the new version this is only added if the SimpleCharacter's
texture property is set. But the texture property is not set.

We could instead set the texture and drop the Sprite2D2 child. However it's not
as trivial as setting the texture property because the sprite is the wrong size
and is scaled down to 0.321 its full size by the Sprite2D2.

So, add a CollisionPolygon2D, and adjust the position of the dead zone
accordingly.
@wjt
Copy link
Copy Markdown
Member Author

wjt commented Oct 17, 2024

We came to the conclusion in discussion today that we should instead remove this demo because the new one from godotcon is a better demo of how to make a runner.

@wjt wjt closed this Oct 17, 2024
@wjt wjt deleted the push-mzluwosknuvt branch October 17, 2024 19:38
@wjt wjt mentioned this pull request Oct 18, 2024
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.

1 participant