Skip to content
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.

Sign up for GitHub

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

Jump to bottom

check collideDown, collideUp, collideLeft, and collideRight in SatSolver #28

Closed
wants to merge 1 commit into from
Closed

check collideDown, collideUp, collideLeft, and collideRight in SatSolver #28

wants to merge 1 commit into from

Conversation

kevin-chau
Copy link

Fixes #27

@hexus
Copy link
Owner

hexus commented Feb 9, 2017

Thanks for the PR, I may implement it slightly differently though, and without all of those whitespace changes. :)

@hexus hexus self-requested a review February 9, 2017 14:34
hexus added a commit that referenced this pull request Feb 9, 2017
@hexus
Supported tile collision flags when determining separation (#27, #28).
5d61fdf 
Fixed collision snapping running when it shouldn't (#21).
@hexus
Copy link
Owner

hexus commented Feb 9, 2017

In the end I implemented it the same way (5d61fdf), just without the whitespace changes.

Thanks!

@hexus hexus closed this Feb 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hexus hexus Awaiting requested review from hexus

Assignees
No one assigned
Labels
compatibility
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kevin-chau @hexus