make Matrix4.makeTranslation accept Vector3

[makc]

Description

This came up after #26031 - Matrix4.setPosition supports both x, y, z and vector signatures, but makeTranslation doesnt

Solution

Why not make it the same as setPosition?

Alternatives

Do nothing.

Additional context

No response

[Mugen87]

I support this change request.

[mrdoob]

Sure 👍

[Mugen87]

@makc Would you like to file a PR with the change?

[makc]

@Mugen87 sure, what do you prefer:

makeTranslation( x, y, z ) {
    return this.identity().setPosition( x, y, z );
}

or should I just copy-pase if-else logic from setPosition?

[Mugen87]

or should I just copy-pase if-else logic from setPosition?

That one.