Skip to content

Conversation

@mstphayassin
Copy link
Contributor

Please check if the PR fulfills these requirements:

  • The commit message follows our guidelines.
  • For bug fixes and features:
    • You tested the changes.

Close issues:

The DrawingTurtle app has been reworked to detect when a user accesses and changes the position variable directly (this was unhandled before). This change fixes major visual bugs that occurred when users accessed both the position variable and used the jump function. It also properly detects when polygons are closed by checking if vertices are aligned. Now, polygons can be closed due to the jump() function, due to a change in the position variable, or when the polygon is actually closed.

I've gone through and tested each and every lesson which uses the DrawingTurtle app and ensured that the correct solutions all still work. However, I can't say that I've gone through and tested every permutation of every error message. I will need some help with that.

One other thing. I've noticed that the camera seems slightly less responsive now. I'm not sure why. I don't believe anything I changed should have affected the camera.

@mstphayassin
Copy link
Contributor Author

mstphayassin commented Nov 17, 2025

Unfortunately, running integration_test_course.tscn results in exactly 1 test failed and 52 tests successful.

However, I don't believe the failed test has anything to do with these commits. I'm getting the same failure message on the latest version.

It has to do with Lesson 2 Practice 1 and does not involve the DrawingTurtle app whatsoever. It is described in issue #1173.

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