Fix look_to resulting in NaN rotations

crates/bevy_transform/src/components/transform.rs

@@ -120,6 +120,10 @@ impl Transform {
 
     /// Returns this [`Transform`] with a new rotation so that [`Transform::forward`]
     /// points towards the `target` position and [`Transform::up`] points towards `up`.
+    ///
+    /// It is not possible to construct a rotation when the resulting forward direction is parallel with `up`.
+    /// If this happens, an orthogonal vector to `up` will be used as the "right" direction to get a valid value.
+    /// It is also not possible when the target is at the same place as the transform. In this case, the Z axis will be used as the "forward" direction.
     #[inline]
     #[must_use]
     pub fn looking_at(mut self, target: Vec3, up: Vec3) -> Self {
@@ -129,6 +133,10 @@ impl Transform {
 
     /// Returns this [`Transform`] with a new rotation so that [`Transform::forward`]
     /// points in the given `direction` and [`Transform::up`] points towards `up`.
+    ///
+    /// It is not possible to construct a rotation when `direction` is parallel with `up`.
+    /// If this happens, an orthogonal vector to `up` will be used as the "right" direction to get a valid value.
+    /// It is also not possible when the direction provided is zero. In this case, the Z axis will be used as the "forward" direction.
     #[inline]
     #[must_use]
     pub fn looking_to(mut self, direction: Vec3, up: Vec3) -> Self {
@@ -325,17 +333,28 @@ impl Transform {
 
     /// Rotates this [`Transform`] so that [`Transform::forward`] points towards the `target` position,
     /// and [`Transform::up`] points towards `up`.
+    ///
+    /// It is not possible to construct a rotation when the resulting forward direction is parallel with `up`.
+    /// If this happens, an orthogonal vector to `up` will be used as the "right" direction to get a valid value.
+    /// It is also not possible when the target is at the same place as the transform. In this case, the Z axis will be used as the "forward" direction.
     #[inline]
     pub fn look_at(&mut self, target: Vec3, up: Vec3) {
         self.look_to(target - self.translation, up);
     }
 
     /// Rotates this [`Transform`] so that [`Transform::forward`] points in the given `direction`
     /// and [`Transform::up`] points towards `up`.
+    ///
+    /// It is not possible to construct a rotation when `direction` is parallel with `up`.
+    /// If this happens, an orthogonal vector to `up` will be used as the "right" direction to get a valid value.
+    /// It is also not possible when the direction provided is zero. In this case, the Z axis will be used as the "forward" direction.
     #[inline]
     pub fn look_to(&mut self, direction: Vec3, up: Vec3) {
-        let forward = -direction.normalize();
-        let right = up.cross(forward).normalize();
+        let forward = -direction.try_normalize().unwrap_or(Vec3::Z);
+        let right = up
+            .cross(forward)
+            .try_normalize()
+            .unwrap_or_else(|| up.any_orthogonal_vector());
         let up = forward.cross(right);
         self.rotation = Quat::from_mat3(&Mat3::from_cols(right, up, forward));
     }