Skip to content

Commit

Permalink
Allow settings options panel on mobile
Browse files Browse the repository at this point in the history
  • Loading branch information
CodeDoctorDE committed Nov 29, 2024
1 parent 8e8478a commit a6a924b
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 21 deletions.
6 changes: 3 additions & 3 deletions app/android/Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,16 @@ 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)
jmespath (~> 1, >= 1.6.1)
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)
Expand Down
17 changes: 3 additions & 14 deletions app/lib/settings/view.dart
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand All @@ -158,7 +147,7 @@ class ViewSettingsPage extends StatelessWidget {
.changeColorToolbarEnabled(value),
title: Text(AppLocalizations.of(context).colorToolbar),
secondary: const PhosphorIcon(PhosphorIconsLight.palette),
)
),
]),
),
),
Expand Down
4 changes: 1 addition & 3 deletions app/lib/views/main.dart
Original file line number Diff line number Diff line change
Expand Up @@ -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(),
Expand Down Expand Up @@ -604,7 +603,6 @@ class _MainBody extends StatelessWidget {
),
),
if (optPos == OptionsPanelPosition.bottom &&
!isMobile &&
currentIndex.hideUi ==
HideState.visible)
const ToolbarView(),
Expand Down
2 changes: 1 addition & 1 deletion metadata/en-US/changelogs/123.txt
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit a6a924b

Please sign in to comment.