Skip to content

Conversation

@EyeDaleHim
Copy link
Contributor

Noticed an issue with scaling and this class, thought I might try to add a fix to it.

Noticed an issue with scaling and this class, thought I might try to add a fix to it.
@Geokureli
Copy link
Member

I'd like to try this change out, can you recommend a demo I should use, or a simple repro that showcases this fix? I haven't used FlxMouseEventManager very much

@Geokureli
Copy link
Member

Geokureli commented Jun 4, 2022

this doesn't seem to be working as intended, also FlxMouseEventManager was changed and this branch doesn't use the latest changes, I can tell because the Sprite arg is capitalized.
https://imgur.com/rf3jp06

you bring up a valid issue that should be fixed, though

Edit: To me, it makes more sense that checkPixelPerfectOverlap should just call sprite.pixelsOverlapPoint which should already be accounting for scale and angle

@Geokureli
Copy link
Member

replaced by #2576

@Geokureli Geokureli closed this Jun 5, 2022
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