From 85e82d0bd57929c1693af766ce3dab82c1997498 Mon Sep 17 00:00:00 2001 From: fufesou Date: Mon, 4 Sep 2023 23:21:47 +0800 Subject: [PATCH] fix, mobile connecting canvas color Signed-off-by: fufesou --- flutter/lib/mobile/pages/remote_page.dart | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/flutter/lib/mobile/pages/remote_page.dart b/flutter/lib/mobile/pages/remote_page.dart index 519e228f8..838545d0c 100644 --- a/flutter/lib/mobile/pages/remote_page.dart +++ b/flutter/lib/mobile/pages/remote_page.dart @@ -92,12 +92,16 @@ class _RemotePageState extends State { removeSharedStates(widget.id); } - Widget emptyOverlay() => BlockableOverlay( + // to-do: It should be better to use transparent color instead of the bgColor. + // But for now, the transparent color will cause the canvas to be white. + // I'm sure that the white color is caused by the Overlay widget in BlockableOverlay. + // But I don't know why and how to fix it. + Widget emptyOverlay(Color bgColor) => BlockableOverlay( /// the Overlay key will be set with _blockableOverlayState in BlockableOverlay /// see override build() in [BlockableOverlay] state: _blockableOverlayState, underlying: Container( - color: Colors.transparent, + color: bgColor, ), ); @@ -265,13 +269,13 @@ class _RemotePageState extends State { children: [ gFFI.ffiModel.pi.isSet.isTrue && gFFI.ffiModel.waitForFirstImage.isTrue - ? emptyOverlay() + ? emptyOverlay(MyTheme.canvasColor) : () { gFFI.ffiModel.tryShowAndroidActionsOverlay(); return Offstage(); }(), _bottomWidget(), - gFFI.ffiModel.pi.isSet.isFalse ? emptyOverlay() : Offstage(), + gFFI.ffiModel.pi.isSet.isFalse ? emptyOverlay(MyTheme.canvasColor) : Offstage(), ], )), body: Overlay(