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

Jump to bottom

Length manhattan for the Vector2. #6786

Merged
merged 3 commits into from Jul 4, 2015
Merged

Length manhattan for the Vector2. #6786

merged 3 commits into from Jul 4, 2015

Conversation

saromanov
Copy link
Contributor

Hi!
It seems that lengthManhattan is missed for the Vector2.
Just added it.

@WestLangley
Copy link
Collaborator

Looks good.

@erichlof
Copy link
Contributor

erichlof commented Jul 4, 2015

Hi @saromanov ,
Manhattan should be with an 'a' at the end rather than an 'e'. Minor spelling point, but anyway thanks for adding this method! :-)

@saromanov
Copy link
Contributor Author

@erichlof , oops :) thank you for this remark!

@saromanov saromanov changed the title Length manhatten for the Vector2. Length manhattan for the Vector2. Jul 4, 2015
mrdoob added a commit that referenced this pull request Jul 4, 2015
@mrdoob
Merge pull request #6786 from saromanov/lengthManhatten-Vector2
99340ec 
Length manhattan for the Vector2.
@mrdoob mrdoob merged commit 99340ec into mrdoob:dev Jul 4, 2015
@mrdoob
Copy link
Owner

mrdoob commented Jul 4, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@saromanov @WestLangley @erichlof @mrdoob