License page now has its own route

ferraridamiano · Feb 4, 2024 · 4f4b8d3 · 4f4b8d3
1 parent 63a4714
commit 4f4b8d3
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/lib/app_router.dart b/lib/app_router.dart
@@ -9,8 +9,12 @@ import 'package:converterpro/pages/settings_page.dart';
 import 'package:converterpro/pages/splash_screen.dart';
 import 'package:converterpro/utils/app_scaffold.dart';
 import 'package:converterpro/utils/utils.dart';
+import 'package:flutter/material.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
+import 'package:flutter_svg/flutter_svg.dart';
 import 'package:go_router/go_router.dart';
+import 'package:translations/app_localizations.dart';
+import 'package:vector_graphics/vector_graphics.dart';
 
 final isEverythingLoadedProvider = Provider<bool>((ref) =>
     ref.watch(SignificantFigures.provider).hasValue &&
@@ -85,6 +89,17 @@ final routerProvider = Provider<GoRouter>(
                   ),
                 ],
               ),
+              GoRoute(
+                path: 'about',
+                name: 'about',
+                builder: (context, state) => LicensePage(
+                  applicationName: AppLocalizations.of(context)!.appName,
+                  applicationIcon: const SvgPicture(
+                    AssetBytesLoader('assets/app_icons/logo.svg.vec'),
+                    width: 50,
+                  ),
+                ),
+              ),
             ],
           ),
         ],

diff --git a/lib/pages/settings_page.dart b/lib/pages/settings_page.dart
@@ -327,12 +327,7 @@ class SettingsPage extends ConsumerWidget {
             style: textStyle,
           ),
           shape: const RoundedRectangleBorder(borderRadius: borderRadius),
-          onTap: () {
-            showLicensePage(
-              context: context,
-              applicationName: AppLocalizations.of(context)!.appName,
-            );
-          },
+          onTap: () => context.goNamed('about'),
         ),
       ].map(ConstrainedContainer.new).toList()))
     ]);