From e34c51cc01a0e019874d4adb5350522383bf452f Mon Sep 17 00:00:00 2001
From: Alex Kocharin <alex@kocharin.ru>
Date: Sun, 14 Apr 2024 22:43:47 +0400
Subject: [PATCH] change camera handling in update_gizmos function

 - support the case where there're no gizmo cameras at all (before this commit, it was a warning every frame)
 - support multiple inactive gizmo cameras (you can now have only 1 active gizmo camera, but multiple inactive ones)
---
 crates/transform-gizmo-bevy/src/lib.rs | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/crates/transform-gizmo-bevy/src/lib.rs b/crates/transform-gizmo-bevy/src/lib.rs
index d966883..25a1bdb 100644
--- a/crates/transform-gizmo-bevy/src/lib.rs
+++ b/crates/transform-gizmo-bevy/src/lib.rs
@@ -177,9 +177,25 @@ fn update_gizmos(
 
     let scale_factor = window.scale_factor();
 
-    let Ok((camera, camera_transform)) = q_gizmo_camera.get_single() else {
-        bevy::log::warn!("Only one camera with a GizmoCamera component is supported.");
-        return;
+    let (camera, camera_transform) = {
+        let mut active_camera = None;
+
+        for camera in q_gizmo_camera.iter() {
+            if !camera.0.is_active {
+                continue;
+            }
+            if active_camera.is_some() {
+                // multiple active cameras found, warn and skip
+                bevy::log::warn!("Only one camera with a GizmoCamera component is supported.");
+                return;
+            }
+            active_camera = Some(camera);
+        }
+
+        match active_camera {
+            Some(camera) => camera,
+            None => return, // no active cameras in the scene
+        }
     };
 
     let Some(viewport) = camera.logical_viewport_rect() else {