From 435ed7bee61f6864cd1fd7654eb0e8178dbff769 Mon Sep 17 00:00:00 2001
From: Nick Fisher <nick.fisher@avinium.com>
Date: Tue, 24 Oct 2023 12:24:56 +1100
Subject: [PATCH] don't use resize callback on Windows and use
 ListenableBuilder for texture ID changes

don't use resize callback on Windows and use ListenableBuilder for texture ID changes
---
 lib/widgets/filament_widget.dart | 52 ++++++++++++++++++--------------
 1 file changed, 29 insertions(+), 23 deletions(-)

diff --git a/lib/widgets/filament_widget.dart b/lib/widgets/filament_widget.dart
index 568ce49a..9191592f 100644
--- a/lib/widgets/filament_widget.dart
+++ b/lib/widgets/filament_widget.dart
@@ -42,7 +42,7 @@ class _RenderResizeObserver extends RenderProxyBox {
   void performLayout() async {
     super.performLayout();
     if (size.width != _oldSize.width || size.height != _oldSize.height) {
-      onLayoutChangedCallback(size);
+        onLayoutChangedCallback(size);
       _oldSize = Size(size.width, size.height);
     }
   }
@@ -87,6 +87,9 @@ class _FilamentWidgetState extends State<FilamentWidget> {
 
     return ResizeObserver(
         onResized: (newSize) {
+          if(!Platform.isWindows) {
+            return;
+          }
           WidgetsBinding.instance.addPostFrameCallback((timeStamp) {
             setState(() {
               _width = newSize.width.ceil();
@@ -249,28 +252,31 @@ class _SizedFilamentWidgetState extends State<_SizedFilamentWidget> {
           ]));
     }
 
-    if (widget.controller.textureDetails == null) { 
-      return Stack(children: [
-        Positioned.fill(child: widget.initial ?? Container(color: Colors.red))
-      ]);
-    }
-    // see [FilamentControllerFFI.resize] for an explanation of how we deal with resizing
-    var texture = Texture(
-      key: ObjectKey("texture_${widget.controller.textureDetails!.textureId}"),
-      textureId: widget.controller.textureDetails!.textureId,
-      filterQuality: FilterQuality.none,
-      freeze: false,
-    );
+    return ListenableBuilder(listenable: widget.controller.textureDetails, builder: (BuildContext ctx, Widget? wdgt) {
 
-    return Stack(children: [
-      Positioned.fill(
-          child: Platform.isLinux || Platform.isWindows
-              ? Transform(
-                  alignment: Alignment.center,
-                  transform: Matrix4.rotationX(
-                      pi), // TODO - this rotation is due to OpenGL texture coordinate working in a different space from Flutter, can we move this to the C++ side somewhere?
-                  child: texture)
-              : texture)
-    ]);
+      if (widget.controller.textureDetails.value == null) { 
+        return Stack(children: [
+          Positioned.fill(child: widget.initial ?? Container(color: Colors.red))
+        ]);
+      }
+      // see [FilamentControllerFFI.resize] for an explanation of how we deal with resizing
+      var texture = Texture(
+        key: ObjectKey("texture_${widget.controller.textureDetails.value!.textureId}"),
+        textureId: widget.controller.textureDetails.value!.textureId,
+        filterQuality: FilterQuality.none,
+        freeze: false,
+      );
+
+      return Stack(children: [
+        Positioned.fill(
+            child: Platform.isLinux || Platform.isWindows
+                ? Transform(
+                    alignment: Alignment.center,
+                    transform: Matrix4.rotationX(
+                        pi), // TODO - this rotation is due to OpenGL texture coordinate working in a different space from Flutter, can we move this to the C++ side somewhere?
+                    child: texture)
+                : texture)
+      ]);
+    });
   }
 }