From 5a7ea87543e4790e5346d8ce844f8460527d2b6d Mon Sep 17 00:00:00 2001
From: tuxsudo <tuxsudo@tux.pizza>
Date: Thu, 22 Feb 2024 15:39:39 -0500
Subject: [PATCH] UI fixes (#1301)

* Remove StandardListSeparator from Security and backup page

* Remove StandardListSeparator from Connection and sync page

* Remove StandardListSeparator from Other settings page
---
 lib/src/screens/settings/connection_sync_page.dart | 7 -------
 lib/src/screens/settings/other_settings_page.dart  | 1 -
 lib/src/screens/settings/security_backup_page.dart | 3 ---
 3 files changed, 11 deletions(-)

diff --git a/lib/src/screens/settings/connection_sync_page.dart b/lib/src/screens/settings/connection_sync_page.dart
index 803a65913..ce27cd104 100644
--- a/lib/src/screens/settings/connection_sync_page.dart
+++ b/lib/src/screens/settings/connection_sync_page.dart
@@ -38,13 +38,11 @@ class ConnectionSyncPage extends BasePage {
             title: S.current.reconnect,
             handler: (context) => _presentReconnectAlert(context),
           ),
-          const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
           if (dashboardViewModel.hasRescan) ...[
             SettingsCellWithArrow(
               title: S.current.rescan,
               handler: (context) => Navigator.of(context).pushNamed(Routes.rescan),
             ),
-            const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
             if (DeviceInfo.instance.isMobile) ...[
               Observer(builder: (context) {
                 return SettingsPickerCell<SyncMode>(
@@ -82,7 +80,6 @@ class ConnectionSyncPage extends BasePage {
                       }
                     });
               }),
-              const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
               Observer(builder: (context) {
                 return SettingsSwitcherCell(
                   title: S.current.sync_all_wallets,
@@ -90,14 +87,12 @@ class ConnectionSyncPage extends BasePage {
                   onValueChange: (_, bool value) => dashboardViewModel.setSyncAll(value),
                 );
               }),
-              const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
             ],
           ],
           SettingsCellWithArrow(
             title: S.current.manage_nodes,
             handler: (context) => Navigator.of(context).pushNamed(Routes.manageNodes),
           ),
-          const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
           Observer(
             builder: (context) {
               if (!dashboardViewModel.hasPowNodes) return const SizedBox();
@@ -108,7 +103,6 @@ class ConnectionSyncPage extends BasePage {
                     title: S.current.manage_pow_nodes,
                     handler: (context) => Navigator.of(context).pushNamed(Routes.managePowNodes),
                   ),
-                  const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
                 ],
               );
             },
@@ -117,7 +111,6 @@ class ConnectionSyncPage extends BasePage {
             WalletConnectTile(
               onTap: () => Navigator.of(context).pushNamed(Routes.walletConnectConnectionsListing),
             ),
-            const StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
           ],
           if (FeatureFlag.isInAppTorEnabled)
             SettingsCellWithArrow(
diff --git a/lib/src/screens/settings/other_settings_page.dart b/lib/src/screens/settings/other_settings_page.dart
index c58523185..c88804147 100644
--- a/lib/src/screens/settings/other_settings_page.dart
+++ b/lib/src/screens/settings/other_settings_page.dart
@@ -61,7 +61,6 @@ class OtherSettingsPage extends BasePage {
                 handler: (BuildContext context) =>
                     Navigator.of(context).pushNamed(Routes.readDisclaimer),
               ),
-              StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
               Spacer(),
               SettingsVersionCell(
                   title: S.of(context).version(_otherSettingsViewModel.currentVersion)),
diff --git a/lib/src/screens/settings/security_backup_page.dart b/lib/src/screens/settings/security_backup_page.dart
index 56ba049ba..e559e9b15 100644
--- a/lib/src/screens/settings/security_backup_page.dart
+++ b/lib/src/screens/settings/security_backup_page.dart
@@ -37,7 +37,6 @@ class SecurityBackupPage extends BasePage {
                   .shouldRequireTOTP2FAForAllSecurityAndBackupSettings,
             ),
           ),
-          StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
           SettingsCellWithArrow(
             title: S.current.create_backup,
             handler: (_) => _authService.authenticateAction(
@@ -47,7 +46,6 @@ class SecurityBackupPage extends BasePage {
                   .shouldRequireTOTP2FAForAllSecurityAndBackupSettings,
             ),
           ),
-          StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
           SettingsCellWithArrow(
             title: S.current.settings_change_pin,
             handler: (_) => _authService.authenticateAction(
@@ -60,7 +58,6 @@ class SecurityBackupPage extends BasePage {
                   .shouldRequireTOTP2FAForAllSecurityAndBackupSettings,
             ),
           ),
-          StandardListSeparator(padding: EdgeInsets.symmetric(horizontal: 24)),
           if (DeviceInfo.instance.isMobile)
             Observer(builder: (_) {
               return SettingsSwitcherCell(