diff --git a/lib/screens/settings/main.dart b/lib/screens/settings/main.dart index c069085..7424bce 100644 --- a/lib/screens/settings/main.dart +++ b/lib/screens/settings/main.dart @@ -112,23 +112,50 @@ class _SettingsPageState extends State { title: const Text('Change Theme'), subtitle: const Text( 'You can change between a light theme, a dark theme and automatic theme selection'), - // NOTE: have the trailing item be a value select widget - trailing: SegmentedButton( - selected: {context.watch()}, - selectedIcon: Icon(context.watch().icon), - showSelectedIcon: true, - multiSelectionEnabled: false, - emptySelectionAllowed: false, - segments: AppTheme.list().map((item) { - return ButtonSegment( - value: item, icon: Icon(item.icon), label: Text(item.name)); - }).toList(), - onSelectionChanged: (item) async { - try { - await item.first.toDisk(); - } catch(_) {} - }, - ), + trailing: const Icon(Icons.chevron_right), + onTap: () { + showDialog( + context: context, + builder: (context) => AlertDialog( + title: const Text('Change Theme'), + content: SingleChildScrollView( + child: Column( + children: [ + const Padding( + padding: EdgeInsets.all(8), + child: Text('Choose your preferred theme'), + ), + SegmentedButton( + selected: {context.watch()}, + selectedIcon: Icon(context.watch().icon), + showSelectedIcon: true, + multiSelectionEnabled: false, + emptySelectionAllowed: false, + segments: AppTheme.list().map((item) { + return ButtonSegment( + value: item, + icon: Icon(item.icon), + label: Text(item.name)); + }).toList(), + onSelectionChanged: (item) async { + try { + await item.first.toDisk(); + } catch (_) {} + }, + ), + ] + ) + ), + actions: [ + FilledButton( + child: const Text('Close'), + onPressed: () { + Navigator.of(context).pop(); + }, + ) + ], + )); + }, ), // change password button @@ -136,10 +163,6 @@ class _SettingsPageState extends State { title: const Text('Change password'), subtitle: const Text('Choose a new password for your account'), onTap: () { - // TODO: show confirm dialog - // NOTE: needs an input field for the current password - // NOTE: might want to show a message explaining, - // that there is no way to reset the password showDialog( context: context, builder: (context) => const ChangePasswordDialog());