From a6a924b985311541accb1a10b2220ff3de8c766a Mon Sep 17 00:00:00 2001
From: CodeDoctorDE <codedoctor@linwood.dev>
Date: Fri, 29 Nov 2024 17:10:26 +0100
Subject: [PATCH] Allow settings options panel on mobile

---
 app/android/Gemfile.lock          |  6 +++---
 app/lib/settings/view.dart        | 17 +++--------------
 app/lib/views/main.dart           |  4 +---
 metadata/en-US/changelogs/123.txt |  2 +-
 4 files changed, 8 insertions(+), 21 deletions(-)

diff --git a/app/android/Gemfile.lock b/app/android/Gemfile.lock
index 731eca591e68..3de6045b75be 100644
--- a/app/android/Gemfile.lock
+++ b/app/android/Gemfile.lock
@@ -10,8 +10,8 @@ GEM
     artifactory (3.0.17)
     atomos (0.1.3)
     aws-eventstream (1.3.0)
-    aws-partitions (1.1011.0)
-    aws-sdk-core (3.213.0)
+    aws-partitions (1.1014.0)
+    aws-sdk-core (3.214.0)
       aws-eventstream (~> 1, >= 1.3.0)
       aws-partitions (~> 1, >= 1.992.0)
       aws-sigv4 (~> 1.9)
@@ -19,7 +19,7 @@ GEM
     aws-sdk-kms (1.96.0)
       aws-sdk-core (~> 3, >= 3.210.0)
       aws-sigv4 (~> 1.5)
-    aws-sdk-s3 (1.172.0)
+    aws-sdk-s3 (1.174.0)
       aws-sdk-core (~> 3, >= 3.210.0)
       aws-sdk-kms (~> 1)
       aws-sigv4 (~> 1.5)
diff --git a/app/lib/settings/view.dart b/app/lib/settings/view.dart
index 6c4cc9ea655e..1e5cb3c13b54 100644
--- a/app/lib/settings/view.dart
+++ b/app/lib/settings/view.dart
@@ -136,19 +136,8 @@ class ViewSettingsPage extends StatelessWidget {
                       leading: const PhosphorIcon(PhosphorIconsLight.archive),
                       title: Text(
                           AppLocalizations.of(context).optionsPanelPosition),
-                      subtitle: Column(
-                        mainAxisSize: MainAxisSize.min,
-                        crossAxisAlignment: CrossAxisAlignment.stretch,
-                        children: [
-                          Text(state.optionsPanelPosition
-                              .getLocalizedName(context)),
-                          Text(
-                            AppLocalizations.of(context)
-                                .onlyAvailableLargerScreen,
-                            style: Theme.of(context).textTheme.labelSmall,
-                          ),
-                        ],
-                      ),
+                      subtitle: Text(
+                          state.optionsPanelPosition.getLocalizedName(context)),
                       onTap: () => _openOptionsPanelPositionModal(context),
                     ),
                     SwitchListTile(
@@ -158,7 +147,7 @@ class ViewSettingsPage extends StatelessWidget {
                           .changeColorToolbarEnabled(value),
                       title: Text(AppLocalizations.of(context).colorToolbar),
                       secondary: const PhosphorIcon(PhosphorIconsLight.palette),
-                    )
+                    ),
                   ]),
             ),
           ),
diff --git a/app/lib/views/main.dart b/app/lib/views/main.dart
index 4da54827e013..d4a5d6eb4ff2 100644
--- a/app/lib/views/main.dart
+++ b/app/lib/views/main.dart
@@ -561,8 +561,7 @@ class _MainBody extends StatelessWidget {
                                         currentIndex.hideUi ==
                                             HideState.visible)
                                       toolbar,
-                                    if ((optPos == OptionsPanelPosition.top ||
-                                            isMobile) &&
+                                    if (optPos == OptionsPanelPosition.top &&
                                         currentIndex.hideUi ==
                                             HideState.visible)
                                       const ToolbarView(),
@@ -604,7 +603,6 @@ class _MainBody extends StatelessWidget {
                                       ),
                                     ),
                                     if (optPos == OptionsPanelPosition.bottom &&
-                                        !isMobile &&
                                         currentIndex.hideUi ==
                                             HideState.visible)
                                       const ToolbarView(),
diff --git a/metadata/en-US/changelogs/123.txt b/metadata/en-US/changelogs/123.txt
index 8ad4f1f1fc1f..200603d455fa 100644
--- a/metadata/en-US/changelogs/123.txt
+++ b/metadata/en-US/changelogs/123.txt
@@ -2,8 +2,8 @@
 * Add save button indicator for autosave ([#757](https://github.com/LinwoodDev/Butterfly/issues/757))
 * Add duplicate layer button
 * Add tool options panel position
-* Separate laser duration in normal duration and hide duration
 * Add path laser animation
+* Separate laser duration in normal duration and hide duration
 * Use long press to move tools on all platforms to improve desktop touch behavior
 * Use sha checksum for assets
 * Separate personalization settings in new view settings