From 64268560892637a44b36609ea1ec3931c5ec1b75 Mon Sep 17 00:00:00 2001
From: alvarosabu <alvaro.saburido@gmail.com>
Date: Mon, 22 Jan 2024 10:11:43 +0100
Subject: [PATCH] chore: tell dont ask fix

---
 playground/src/pages/rendering-modes/index.vue  | 2 +-
 playground/src/pages/rendering-modes/scene.vue  | 2 +-
 src/composables/useTresContextProvider/index.ts | 6 +++++-
 src/core/nodeOps.ts                             | 2 +-
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/playground/src/pages/rendering-modes/index.vue b/playground/src/pages/rendering-modes/index.vue
index 7c4beb53e..014366a1a 100644
--- a/playground/src/pages/rendering-modes/index.vue
+++ b/playground/src/pages/rendering-modes/index.vue
@@ -13,7 +13,7 @@ setTimeout(() => {
 <template>
   <TresCanvas
     :clear-color="clearColor"
-    render-mode="manual"
+    render-mode="on-demand"
     @render="() => console.log('onRender')"
   >
     <Scene />
diff --git a/playground/src/pages/rendering-modes/scene.vue b/playground/src/pages/rendering-modes/scene.vue
index c076335c2..c44e5d194 100644
--- a/playground/src/pages/rendering-modes/scene.vue
+++ b/playground/src/pages/rendering-modes/scene.vue
@@ -5,7 +5,7 @@ import { OrbitControls } from '@tresjs/cientos'
 const { invalidate, advance } = useTres()
 
 function onControlChange() {
-  advance()
+  invalidate()
 }
 
 const positionX = ref(0)
diff --git a/src/composables/useTresContextProvider/index.ts b/src/composables/useTresContextProvider/index.ts
index 7828f9bf6..935ba527d 100644
--- a/src/composables/useTresContextProvider/index.ts
+++ b/src/composables/useTresContextProvider/index.ts
@@ -45,6 +45,7 @@ export interface TresContext {
    * If set to 'always', the scene will be rendered every frame
    */
   renderMode: Ref<'always' | 'on-demand' | 'manual'>
+  canBeInvalidated: ComputedRef<boolean>
   internal: InternalState
   /**
    * Invalidates the current frame when renderMode === 'on-demand'
@@ -147,6 +148,8 @@ export function useTresContextProvider({
       disableRender,
     })
 
+  const renderMode = ref<'always' | 'on-demand' | 'manual'>(rendererOptions.renderMode || 'always')
+
   const toProvide: TresContext = {
     sizes,
     scene: localScene,
@@ -167,7 +170,8 @@ export function useTresContextProvider({
         accumulator: [],
       },
     },
-    renderMode: ref(rendererOptions.renderMode || 'always'),
+    renderMode,
+    canBeInvalidated: computed(() => renderMode.value === 'on-demand' && internal.frames.value === 0),
     internal,
     advance,
     extend,
diff --git a/src/core/nodeOps.ts b/src/core/nodeOps.ts
index 814d80574..8301306b0 100644
--- a/src/core/nodeOps.ts
+++ b/src/core/nodeOps.ts
@@ -28,7 +28,7 @@ export function invalidateInstance(instance: TresObject) {
   
   if (!ctx) return
   
-  if (ctx.renderMode.value === 'on-demand' && ctx.internal.frames.value === 0) {
+  if (ctx.canBeInvalidated.value) {
     ctx.invalidate()
   }