From aa9a6f56b653c0a36aab289b7c39df7fce0777d1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicol=C3=B2=20P=2E?= <nicolo.paraciani@cnr.it>
Date: Tue, 19 May 2026 07:49:40 +0200
Subject: [PATCH] clipping: Fix wrong AO handling

---
 js/controllers/clipper_controller.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/js/controllers/clipper_controller.js b/js/controllers/clipper_controller.js
index 07551a2..3eb34d9 100644
--- a/js/controllers/clipper_controller.js
+++ b/js/controllers/clipper_controller.js
@@ -56,16 +56,16 @@ export default class extends Controller {
             btn.classList.remove('border', 'border-2', 'border-warning');
         });
 
-        // AO should be turned off if clipping is enabled
-        toggleAmbientOcclusion(!this.enabledValue);
-
         if (this.enabledValue) {
+            // AO should be turned off if clipping is enabled
+            toggleAmbientOcclusion(false);
             trigger.className += ' border border-2 border-white';
         }
 
         if (!this.enabledValue) {
             resetClipping();
             trigger.className = trigger.className.replace(/ border.*$/g, '');
+            toggleAmbientOcclusion(AppState.ambientOcclusion);
         }
 
         AppState.clipping.enabled = this.enabledValue;