Render whenever button_x has changed

OneZoom · Nov 13, 2024 · 24783ac · 24783ac
1 parent 37d081b
commit 24783ac
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/OZprivate/rawJS/OZTreeModule/src/interactor/touch.js b/OZprivate/rawJS/OZTreeModule/src/interactor/touch.js
@@ -33,9 +33,6 @@ class TouchInteractor {
     } else if (event.targetTouches.length == 1) {      
       this.clicking = true;
     }
-    // Jog the tree slightly to ensure that node hover-over states update
-    // NB: Obviously a hack, ideally we'd force this update without - see #155
-    this.controller.pan(1, 0);
   }
 
   touch_move(event) {

diff --git a/OZprivate/rawJS/OZTreeModule/src/render/renderer.js b/OZprivate/rawJS/OZTreeModule/src/render/renderer.js
@@ -11,6 +11,8 @@ let last_yp = null;
 let last_ws = null;
 let last_btn_data = null;
 let last_btn_action = null;
+let last_button_x = null;
+let last_button_y = null;
 
 //Do not skip refresh when render_id = 60, 120, 180... because we want to refresh page when node details or images get fetched.
 let render_id = 0;
@@ -110,6 +112,11 @@ function need_refresh() {
 
   if (render_id % 60 === 0) return true;
   if (tree_state.xp != last_xp || tree_state.yp != last_yp || tree_state.ws != last_ws) return true;
+  if (tree_state.button_x != last_button_x || tree_state.button_y != last_button_y) {
+    last_button_x = tree_state.button_x;
+    last_button_y = tree_state.button_y;
+    return true;
+  }
   if (!areEqual(global_button_action.action,last_btn_action)) return true
   if (!areEqual(global_button_action.data,last_btn_data)) return true;
   return false;

diff --git a/OZprivate/rawJS/OZTreeModule/src/tree_state.js b/OZprivate/rawJS/OZTreeModule/src/tree_state.js
@@ -28,6 +28,8 @@ class TreeState {
     this.url_parsed = false;
     this.last_active_at = new Date();
     this.last_render_at = new Date();
+    this.button_x = null;
+    this.button_y = null;
     let self = this;
     setTimeout(function() {
       self.url_parsed = true;