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.

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

Update docs to use Move instead of Moved #1173

Merged
merged 1 commit into from
Jan 27, 2018

Conversation

starwed
Copy link
Member

@starwed starwed commented Jan 26, 2018

This should address the misleading documentation mentioned here.

@starwed starwed added the docs label Jan 26, 2018
@matthijsgroen
Copy link
Member

Trick with this (for me at least) was if a movement is heading south-east, and the entity has a collision only on the south side, to keep the motion eastward intact (gliding over the other entity). with reverting both axis, the entity gets 'stuck' when going south-east

@starwed
Copy link
Member Author

starwed commented Jan 27, 2018

That's true, but in terms of just updating this example I think the simpler collision resolution is fine.

e: Probably we should have a general doc section on collision management that explains these things.

@starwed starwed merged commit f4a79d7 into craftyjs:develop Jan 27, 2018
@starwed starwed deleted the remove-moved-from-docs branch January 27, 2018 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants