From 9cdd0ac6d064820a5c4f59dccc327292143e204c Mon Sep 17 00:00:00 2001
From: howard-stearns <howard.stearns@gmail.com>
Date: Fri, 11 Mar 2016 10:19:44 -0800
Subject: [PATCH] Respect zone visibility property.

---
 libraries/entities-renderer/src/EntityTreeRenderer.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libraries/entities-renderer/src/EntityTreeRenderer.cpp b/libraries/entities-renderer/src/EntityTreeRenderer.cpp
index 5cf1b1548a..180015b034 100644
--- a/libraries/entities-renderer/src/EntityTreeRenderer.cpp
+++ b/libraries/entities-renderer/src/EntityTreeRenderer.cpp
@@ -364,8 +364,9 @@ void EntityTreeRenderer::applyZonePropertiesToScene(std::shared_ptr<ZoneEntityIt
                     _pendingSkyboxTexture = true;
                 }
             }
-
-            skyStage->setBackgroundMode(model::SunSkyStage::SKY_BOX);
+            // Visibility does not effect other side effects such as ambient light or the selection of "best" zone.
+            // The skyStage backgroundMode is the hammer used (e.g., in Application.cpp) to control the visiblity of the sky box.
+            skyStage->setBackgroundMode(zone->getVisible() ? model::SunSkyStage::SKY_BOX : model::SunSkyStage::SKY_DOME);
             break;
         }