hongshaorou/rustdesk
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: tabbar rebuild issue

Browse Source 
Signed-off-by: Kingtous <kingtous@qq.com>
This commit is contained in:
Kingtous 2022-08-05 10:27:06 +08:00
parent ab6a83e8b0
commit 1977ee951e
3 changed files with 132 additions and 123 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

60
flutter/lib/desktop/pages/connection_tab_page.dart
View File

@ -7,7 +7,6 @@ import 'package:flutter_hbb/consts.dart';
import 'package:flutter_hbb/desktop/pages/remote_page.dart'; import 'package:flutter_hbb/desktop/pages/remote_page.dart';
import 'package:flutter_hbb/desktop/widgets/titlebar_widget.dart'; import 'package:flutter_hbb/desktop/widgets/titlebar_widget.dart';
import 'package:flutter_hbb/utils/multi_window_manager.dart'; import 'package:flutter_hbb/utils/multi_window_manager.dart';
import 'package:provider/provider.dart';
import 'package:get/get.dart'; import 'package:get/get.dart';
import '../../models/model.dart'; import '../../models/model.dart';
@ -22,11 +21,14 @@ class ConnectionTabPage extends StatefulWidget {
} }
class _ConnectionTabPageState extends State<ConnectionTabPage> class _ConnectionTabPageState extends State<ConnectionTabPage>
with SingleTickerProviderStateMixin { with TickerProviderStateMixin {
// refactor List<int> when using multi-tab // refactor List<int> when using multi-tab
// this singleton is only for test // this singleton is only for test
List<String> connectionIds = List.empty(growable: true); var connectionIds = RxList.empty(growable: true);
var initialIndex = 0; var initialIndex = 0;
late Rx<TabController> tabController;
var connectionMap = RxList<Widget>.empty(growable: true);
_ConnectionTabPageState(Map<String, dynamic> params) { _ConnectionTabPageState(Map<String, dynamic> params) {
if (params['id'] != null) { if (params['id'] != null) {
@ -37,26 +39,27 @@ class _ConnectionTabPageState extends State<ConnectionTabPage>
@override @override
void initState() { void initState() {
super.initState(); super.initState();
tabController =
TabController(length: connectionIds.length, vsync: this).obs;
rustDeskWinManager.setMethodHandler((call, fromWindowId) async { rustDeskWinManager.setMethodHandler((call, fromWindowId) async {
print( print(
"call ${call.method} with args ${call.arguments} from window ${fromWindowId}"); "call ${call.method} with args ${call.arguments} from window ${fromWindowId}");
// for simplify, just replace connectionId // for simplify, just replace connectionId
if (call.method == "new_remote_desktop") { if (call.method == "new_remote_desktop") {
setState(() {
final args = jsonDecode(call.arguments); final args = jsonDecode(call.arguments);
final id = args['id']; final id = args['id'];
final indexOf = connectionIds.indexOf(id); final indexOf = connectionIds.indexOf(id);
if (indexOf >= 0) { if (indexOf >= 0) {
setState(() {
initialIndex = indexOf; initialIndex = indexOf;
}); tabController.value.animateTo(initialIndex, duration: Duration.zero);
} else { } else {
connectionIds.add(id); connectionIds.add(id);
setState(() {
initialIndex = connectionIds.length - 1; initialIndex = connectionIds.length - 1;
}); tabController.value = TabController(
length: connectionIds.length,
vsync: this,
initialIndex: initialIndex);
} }
});
} else if (call.method == "onDestroy") { } else if (call.method == "onDestroy") {
print("executing onDestroy hook, closing ${connectionIds}"); print("executing onDestroy hook, closing ${connectionIds}");
connectionIds.forEach((id) { connectionIds.forEach((id) {
@ -72,11 +75,18 @@ class _ConnectionTabPageState extends State<ConnectionTabPage>
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
final tabBar = TabBar( return Scaffold(
body: Column(
children: [
DesktopTitleBar(
child: Container(
height: kDesktopRemoteTabBarHeight,
child: Obx(() => TabBar(
isScrollable: true, isScrollable: true,
labelColor: Colors.white, labelColor: Colors.white,
physics: NeverScrollableScrollPhysics(), physics: NeverScrollableScrollPhysics(),
indicatorColor: Colors.white, indicatorColor: Colors.white,
controller: tabController.value,
tabs: connectionIds tabs: connectionIds
.map((e) => Tab( .map((e) => Tab(
child: Row( child: Row(
@ -98,30 +108,20 @@ class _ConnectionTabPageState extends State<ConnectionTabPage>
], ],
), ),
)) ))
.toList()); .toList()))),
final tabBarView = TabBarView( ),
Expanded(
child: Obx(() => TabBarView(
controller: tabController.value,
children: connectionIds children: connectionIds
.map((e) => Container( .map((e) => RemotePage(
child: RemotePage(
key: ValueKey(e), key: ValueKey(e),
id: e, id: e,
tabBarHeight: kDesktopRemoteTabBarHeight, tabBarHeight: kDesktopRemoteTabBarHeight,
))) //RemotePage(key: ValueKey(e), id: e)) )) //RemotePage(key: ValueKey(e), id: e))
.toList()); .toList()))),
return Scaffold(
body: DefaultTabController(
initialIndex: initialIndex,
length: connectionIds.length,
animationDuration: Duration.zero,
child: Column(
children: [
DesktopTitleBar(
child: Container(height: kDesktopRemoteTabBarHeight, child: tabBar),
),
Expanded(child: tabBarView),
], ],
), ),
),
); );
} }
@ -130,9 +130,9 @@ class _ConnectionTabPageState extends State<ConnectionTabPage>
if (indexOf == -1) { if (indexOf == -1) {
return; return;
} }
setState(() {
connectionIds.removeAt(indexOf); connectionIds.removeAt(indexOf);
initialIndex = max(0, initialIndex - 1); initialIndex = max(0, initialIndex - 1);
}); tabController.value = TabController(
length: connectionIds.length, vsync: this, initialIndex: initialIndex);
} }
} }

45
flutter/lib/desktop/pages/file_manager_tab_page.dart
View File

@ -20,11 +20,12 @@ class FileManagerTabPage extends StatefulWidget {
} }
class _FileManagerTabPageState extends State<FileManagerTabPage> class _FileManagerTabPageState extends State<FileManagerTabPage>
with SingleTickerProviderStateMixin { with TickerProviderStateMixin {
// refactor List<int> when using multi-tab // refactor List<int> when using multi-tab
// this singleton is only for test // this singleton is only for test
var connectionIds = List<String>.empty(growable: true).obs; var connectionIds = List<String>.empty(growable: true).obs;
var initialIndex = 0.obs; var initialIndex = 0;
late Rx<TabController> tabController;
_FileManagerTabPageState(Map<String, dynamic> params) { _FileManagerTabPageState(Map<String, dynamic> params) {
if (params['id'] != null) { if (params['id'] != null) {
@ -35,6 +36,8 @@ class _FileManagerTabPageState extends State<FileManagerTabPage>
@override @override
void initState() { void initState() {
super.initState(); super.initState();
tabController =
TabController(length: connectionIds.length, vsync: this).obs;
rustDeskWinManager.setMethodHandler((call, fromWindowId) async { rustDeskWinManager.setMethodHandler((call, fromWindowId) async {
print( print(
"call ${call.method} with args ${call.arguments} from window ${fromWindowId}"); "call ${call.method} with args ${call.arguments} from window ${fromWindowId}");
@ -44,10 +47,15 @@ class _FileManagerTabPageState extends State<FileManagerTabPage>
final id = args['id']; final id = args['id'];
final indexOf = connectionIds.indexOf(id); final indexOf = connectionIds.indexOf(id);
if (indexOf >= 0) { if (indexOf >= 0) {
initialIndex.value = indexOf; initialIndex = indexOf;
tabController.value.animateTo(initialIndex, duration: Duration.zero);
} else { } else {
connectionIds.add(id); connectionIds.add(id);
initialIndex.value = connectionIds.length - 1; initialIndex = connectionIds.length - 1;
tabController.value = TabController(
length: connectionIds.length,
initialIndex: initialIndex,
vsync: this);
} }
} else if (call.method == "onDestroy") { } else if (call.method == "onDestroy") {
print("executing onDestroy hook, closing ${connectionIds}"); print("executing onDestroy hook, closing ${connectionIds}");
@ -65,21 +73,19 @@ class _FileManagerTabPageState extends State<FileManagerTabPage>
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
return Scaffold( return Scaffold(
body: Obx( body: Column(
()=> DefaultTabController(
initialIndex: initialIndex.value,
length: connectionIds.length,
animationDuration: Duration.zero,
child: Column(
children: [ children: [
DesktopTitleBar( DesktopTitleBar(
child: TabBar( child: Obx(
() => TabBar(
controller: tabController.value,
isScrollable: true, isScrollable: true,
labelColor: Colors.white, labelColor: Colors.white,
physics: NeverScrollableScrollPhysics(), physics: NeverScrollableScrollPhysics(),
indicatorColor: Colors.white, indicatorColor: Colors.white,
tabs: connectionIds tabs: connectionIds
.map((e) => Tab( .map((e) => Tab(
key: Key('T$e'),
child: Row( child: Row(
mainAxisSize: MainAxisSize.min, mainAxisSize: MainAxisSize.min,
crossAxisAlignment: CrossAxisAlignment.center, crossAxisAlignment: CrossAxisAlignment.center,
@ -101,19 +107,20 @@ class _FileManagerTabPageState extends State<FileManagerTabPage>
)) ))
.toList()), .toList()),
), ),
),
Expanded( Expanded(
child: TabBarView( child: Obx(
() => TabBarView(
controller: tabController.value,
children: connectionIds children: connectionIds
.map((e) => Container( .map((e) => FileManagerPage(
child: FileManagerPage(
key: ValueKey(e), key: ValueKey(e),
id: e))) //RemotePage(key: ValueKey(e), id: e)) id: e)) //RemotePage(key: ValueKey(e), id: e))
.toList()), .toList()),
),
) )
], ],
), ),
),
),
); );
} }
@ -123,6 +130,8 @@ class _FileManagerTabPageState extends State<FileManagerTabPage>
return; return;
} }
connectionIds.removeAt(indexOf); connectionIds.removeAt(indexOf);
initialIndex.value = max(0, initialIndex.value - 1); initialIndex = max(0, initialIndex - 1);
tabController.value = TabController(
length: connectionIds.length, initialIndex: initialIndex, vsync: this);
} }
} }

2
flutter/lib/desktop/pages/remote_page.dart
View File

@ -77,7 +77,7 @@ class _RemotePageState extends State<RemotePage>
@override @override
void dispose() { void dispose() {
print("remote page dispose"); print("REMOTE PAGE dispose ${widget.id}");
hideMobileActionsOverlay(); hideMobileActionsOverlay();
_ffi.listenToMouse(false); _ffi.listenToMouse(false);
_ffi.invokeMethod("enable_soft_keyboard", true); _ffi.invokeMethod("enable_soft_keyboard", true);