-
Notifications
You must be signed in to change notification settings - Fork 39
Fixed sorting issues with layers #20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
| collideHandler(object1, object2, collideCallback, processCallback, callbackContext, overlapOnly) { | ||
| // Only collide valid objects | ||
| if (!object2 && object1.type === Phaser.GROUP) { | ||
| if (object1.type instanceof Phaser.GameObjects.Group && !object2) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
object1.type instanceof Phaser.GameObjects.Group && !object2
This should be
object1 instanceof Phaser.GameObjects.Group && !object2
same with ln 751
| } | ||
| // GROUPS | ||
| else if (object1.type === Phaser.GROUP) { | ||
| else if (object1.type instanceof Phaser.GameObjects.Group) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
object1.type instanceof Phaser.GameObjects.Group
should be
object1 instanceof Phaser.GameObjects.Group
BrennanTanner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove .type from object on line 736 & line 751
Those probably worked in the past. I think I'll officially take on the maintenance of this plugin. There are many features that's been broken by updates. Plus I'll also fix it to work with Phaser's new features. It'll take a while but I think when I'm ready I'll put a comment here leading to my repo so people know to use that instead of this. |
I'm sorry if this is coming lat, but this a fix I made about 2 years back.