[timetable] push "/timetable/palettes/custom" after adding generated …

…palette
plum-tech · Sep 24, 2024 · 5050020 · 5050020
1 parent 84c2605
commit 5050020
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/lib/timetable/p13n/page/background.dart b/lib/timetable/p13n/page/background.dart
@@ -139,12 +139,14 @@ class _TimetableBackgroundEditorState extends ConsumerState<TimetableBackgroundE
               icon: const Icon(Icons.generating_tokens_outlined),
               label: i18n.p13n.background.generatePalette.text(),
               onPressed: () async {
-                await addPaletteFromImageByGenerator(
+                final added = await addPaletteFromImageByGenerator(
                   context,
                   FileImage(renderImageFile),
                 );
+                if (added) {
                 if (!context.mounted) return;
-                context.push("/timetable/palettes/custom");
+                  context.push("/timetable/palettes/custom");
+                }
               },
             ),
         ],

diff --git a/lib/timetable/p13n/utils/generate.dart b/lib/timetable/p13n/utils/generate.dart
@@ -38,7 +38,7 @@ Future<TimetablePalette> generatePaletteFromImage(
   return palette;
 }
 
-Future<void> addPaletteFromImageByGenerator(
+Future<bool> addPaletteFromImageByGenerator(
   BuildContext context,
   ImageProvider img, {
   int maxColorCount = 12,
@@ -47,13 +47,13 @@ Future<void> addPaletteFromImageByGenerator(
     img,
     maxColorCount: maxColorCount,
   );
-  if (!context.mounted) return;
+  if (!context.mounted) return false;
   final newPalette = await context.showSheet<TimetablePalette>(
     (ctx) => TimetablePaletteEditorPage(palette: palette),
     dismissible: false,
   );
-  if (newPalette == null) return;
+  if (newPalette == null) return false;
   TimetableInit.storage.palette.add(newPalette);
   await HapticFeedback.mediumImpact();
-  if (!context.mounted) return;
+  return true;
 }