Do you think the Quaternion.operator* description could be rephrased?

[asperatology]

Here’s the linked article.

This is the description in question:

Combines rotations lhs and rhs.

One suggestion made by another user said this:

Take the rotation state of lhs and apply a rotation of rhs.

Mentioning the reasoning as:

The page describes that it’s equivalent to doing one then the other, but that’s just a different way to say that it’s multiplying the quaternions together. It might be defining multiplication in reverse so lhs * rhs (Unity’s Quatornion.operator*) is the same as rhs * lhs (Actual math multiplication).

[duck]

Yes I see your point, this needs describing in more detail and with less ambiguity because rotation multiplication is not commutatitve (i.e. the order matters!). I’ll clarify the docs. Thanks for the feedback!

[duck]

The summary has been updated. The fix is now online! (the description inside the page already mentioned the non-commutative nature of the operations).

[asperatology]

@duck

The summary has been updated. The fix is now online! (the description inside the page already mentioned the non-commutative nature of the operations).

Now it has two Descriptions and Parameters. An error?

[duck]

No, it’s because you can multiple a quaternion * quaternion, and a point * quaternion. These each have separate signatures, so they have a separate section. (However… I noticed now that the parameters of the 2nd signature have no description! I’ll fix this for the next update )