From cbc9a512ed71126a71a30d8836d481d57c11511d Mon Sep 17 00:00:00 2001 From: Alessandro De Blasis Date: Wed, 29 Oct 2025 10:08:19 +0100 Subject: [PATCH] chore: keep remote_toolbar.dart cleanup (remove dead code) The dead code removed in 265d08fc3 hasn't been used since Aug 2023. Only reverting toolbar.dart is needed for the mobile Scale custom fix. --- .../lib/desktop/widgets/remote_toolbar.dart | 129 ------------------ 1 file changed, 129 deletions(-) diff --git a/flutter/lib/desktop/widgets/remote_toolbar.dart b/flutter/lib/desktop/widgets/remote_toolbar.dart index 1458169c4..84b741d00 100644 --- a/flutter/lib/desktop/widgets/remote_toolbar.dart +++ b/flutter/lib/desktop/widgets/remote_toolbar.dart @@ -153,135 +153,6 @@ class _ToolbarTheme { typedef DismissFunc = void Function(); class RemoteMenuEntry { - static MenuEntryRadios viewStyle( - String remoteId, - FFI ffi, - EdgeInsets padding, { - DismissFunc? dismissFunc, - DismissCallback? dismissCallback, - RxString? rxViewStyle, - }) { - return MenuEntryRadios( - text: translate('Ratio'), - optionsGetter: () => [ - MenuEntryRadioOption( - text: translate('Scale original'), - value: kRemoteViewStyleOriginal, - dismissOnClicked: true, - dismissCallback: dismissCallback, - ), - MenuEntryRadioOption( - text: translate('Scale adaptive'), - value: kRemoteViewStyleAdaptive, - dismissOnClicked: true, - dismissCallback: dismissCallback, - ), - MenuEntryRadioOption( - text: translate('Scale custom'), - value: kRemoteViewStyleCustom, - dismissOnClicked: true, - dismissCallback: dismissCallback, - ), - ], - curOptionGetter: () async { - // null means peer id is not found, which there's no need to care about - final viewStyle = - await bind.sessionGetViewStyle(sessionId: ffi.sessionId) ?? ''; - if (rxViewStyle != null) { - rxViewStyle.value = viewStyle; - } - return viewStyle; - }, - optionSetter: (String oldValue, String newValue) async { - await bind.sessionSetViewStyle( - sessionId: ffi.sessionId, value: newValue); - if (rxViewStyle != null) { - rxViewStyle.value = newValue; - } - ffi.canvasModel.updateViewStyle(); - if (dismissFunc != null) { - dismissFunc(); - } - }, - padding: padding, - dismissOnClicked: true, - dismissCallback: dismissCallback, - ); - } - - static MenuEntrySwitch2 showRemoteCursor( - String remoteId, - SessionID sessionId, - EdgeInsets padding, { - DismissFunc? dismissFunc, - DismissCallback? dismissCallback, - }) { - final state = ShowRemoteCursorState.find(remoteId); - final optKey = 'show-remote-cursor'; - return MenuEntrySwitch2( - switchType: SwitchType.scheckbox, - text: translate('Show remote cursor'), - getter: () { - return state; - }, - setter: (bool v) async { - await bind.sessionToggleOption(sessionId: sessionId, value: optKey); - state.value = - bind.sessionGetToggleOptionSync(sessionId: sessionId, arg: optKey); - if (dismissFunc != null) { - dismissFunc(); - } - }, - padding: padding, - dismissOnClicked: true, - dismissCallback: dismissCallback, - ); - } - - static MenuEntrySwitch disableClipboard( - SessionID sessionId, - EdgeInsets? padding, { - DismissFunc? dismissFunc, - DismissCallback? dismissCallback, - }) { - return createSwitchMenuEntry( - sessionId, - 'Disable clipboard', - 'disable-clipboard', - padding, - true, - dismissCallback: dismissCallback, - ); - } - - static MenuEntrySwitch createSwitchMenuEntry( - SessionID sessionId, - String text, - String option, - EdgeInsets? padding, - bool dismissOnClicked, { - DismissFunc? dismissFunc, - DismissCallback? dismissCallback, - }) { - return MenuEntrySwitch( - switchType: SwitchType.scheckbox, - text: translate(text), - getter: () async { - return bind.sessionGetToggleOptionSync( - sessionId: sessionId, arg: option); - }, - setter: (bool v) async { - await bind.sessionToggleOption(sessionId: sessionId, value: option); - if (dismissFunc != null) { - dismissFunc(); - } - }, - padding: padding, - dismissOnClicked: dismissOnClicked, - dismissCallback: dismissCallback, - ); - } - static MenuEntryButton insertLock( SessionID sessionId, EdgeInsets? padding, {