✨ After review changes

frontend/src/app/render_wasm/shape.cljs

@@ -259,10 +259,13 @@
           (api/update-shape-tiles)
           (api/request-render "set-wasm-attrs")))))
 
+;; `conj` empty set initialization
+(def conj* (fnil conj #{}))
+
 (defn- impl-assoc
   [self k v]
   (when shape/*shape-changes*
-    (vswap! shape/*shape-changes* update (:id self) (fnil conj #{}) k))
+    (vswap! shape/*shape-changes* update (:id self) conj* k))
 
   (case k
     :id
@@ -284,13 +287,8 @@
 
 (defn- impl-dissoc
   [self k]
-  #_(when ^boolean shape/*wasm-sync*
-      (binding [shape/*wasm-sync* false]
-        (when (shape-in-current-page? (.-id ^ShapeProxy self))
-          (set-wasm-attrs! self k nil))))
-
   (when shape/*shape-changes*
-    (vswap! shape/*shape-changes* update (:id self) (fnil conj #{}) k))
+    (vswap! shape/*shape-changes* update (:id self) conj* k))
 
   (case k
     :id

render-wasm/src/shapes.rs

@@ -1151,7 +1151,7 @@ impl Shape {
     pub fn apply_transform(&mut self, transform: &Matrix) {
         self.transform_selrect(transform);
 
-        // We don't need to invalidate this? we can just transform it
+        // TODO: See if we can change this invalidation to a transformation
         self.invalidate_extrect();
         self.invalidate_bounds();
 

render-wasm/src/state/shapes_pool.rs

@@ -266,8 +266,6 @@ impl<'a> ShapesPoolImpl<'a> {
     }
 
     pub fn set_modifiers(&mut self, modifiers: HashMap<Uuid, skia::Matrix>) {
-        // self.clean_shape_cache();
-
         // Convert HashMap<Uuid, V> to HashMap<&'a Uuid, V> using references from shapes and
         // Initialize the cache cells because later we don't want to have the mutable pointer
 
@@ -394,25 +392,3 @@ impl<'a> ShapesPoolImpl<'a> {
         shape.is_bool()
     }
 }
-
-// fn is_modified_child(
-//     shape: &Shape,
-//     shapes: ShapesPoolRef,
-//     modifiers: &HashMap<Uuid, Matrix>,
-// ) -> bool {
-//     if modifiers.is_empty() {
-//         return false;
-//     }
-//
-//     let ids = shape.all_children(shapes, true, false);
-//     let default = &Matrix::default();
-//     let parent_modifier = modifiers.get(&shape.id).unwrap_or(default);
-//
-//     // Returns true if the transform of any child is different to the parent's
-//     ids.iter().any(|id| {
-//         !math::is_close_matrix(
-//             parent_modifier,
-//             modifiers.get(id).unwrap_or(&Matrix::default()),
-//         )
-//     })
-// }