Remove the unnecessary setState, as RxBool is used

Signed-off-by: dignow <linlong1265@gmail.com>
This commit is contained in:
dignow 2023-10-17 14:42:35 +08:00
parent b2404809fc
commit fdfeec54d7

View File

@ -353,7 +353,8 @@ class _RemoteToolbarState extends State<RemoteToolbar> {
void _setFullscreen(bool v) { void _setFullscreen(bool v) {
stateGlobal.setFullscreen(v); stateGlobal.setFullscreen(v);
setState(() {}); // stateGlobal.fullscreen is RxBool now, no need to call setState.
// setState(() {});
} }
RxBool get show => widget.state.show; RxBool get show => widget.state.show;
@ -2058,32 +2059,34 @@ class _DraggableShowHideState extends State<_DraggableShowHide> {
mainAxisSize: MainAxisSize.min, mainAxisSize: MainAxisSize.min,
children: [ children: [
_buildDraggable(context), _buildDraggable(context),
Obx(()=>TextButton( Obx(() => TextButton(
onPressed: () { onPressed: () {
widget.setFullscreen(!isFullscreen.value); widget.setFullscreen(!isFullscreen.value);
setState(() {}); },
}, child: Tooltip(
child: Tooltip( message: translate(
message: translate(isFullscreen.isTrue ? 'Exit Fullscreen' : 'Fullscreen'), isFullscreen.isTrue ? 'Exit Fullscreen' : 'Fullscreen'),
child: Icon( child: Icon(
isFullscreen.isTrue ? Icons.fullscreen_exit : Icons.fullscreen, isFullscreen.isTrue
size: iconSize, ? Icons.fullscreen_exit
), : Icons.fullscreen,
), size: iconSize,
)), ),
Offstage(
offstage: !isFullscreen.value,
child: TextButton(
onPressed: () => widget.setMinimize(),
child: Tooltip(
message: translate('Minimize'),
child: Icon(
Icons.remove,
size: iconSize,
), ),
), )),
), Obx(() => Offstage(
), offstage: isFullscreen.isFalse,
child: TextButton(
onPressed: () => widget.setMinimize(),
child: Tooltip(
message: translate('Minimize'),
child: Icon(
Icons.remove,
size: iconSize,
),
),
),
)),
TextButton( TextButton(
onPressed: () => setState(() { onPressed: () => setState(() {
widget.show.value = !widget.show.value; widget.show.value = !widget.show.value;