Skip to content

Conversation

@Geokureli
Copy link
Member

@Geokureli Geokureli commented Dec 1, 2025

Alternative to #3514 (Most code was copied from this)

Changing a point's x should trigger the x and xy callback, once (if non-null), same with y, any method that changes both should call all 3 callbacks, once.

  • The y callback arg was never being used, so setting xy called the xy callback twice.
  • Every operation function now calls set() rather than setting x and y individually.
  • Setting x and y individually now calls the xy callback as well.
  • Added a unit test to verify the number of sets for each method with each number of callbacks

@Geokureli
Copy link
Member Author

CI is failing because of HaxeFoundation/hscript#155

@Geokureli Geokureli added this to the Next Patch milestone Dec 1, 2025
@Geokureli Geokureli merged commit 524a31b into HaxeFlixel:dev Dec 1, 2025
5 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant