Handle Hacker News links routing in-app

lib/auth/cubit/auth_cubit.dart

@@ -1,6 +1,7 @@
 import 'package:bloc/bloc.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter_inappwebview/flutter_inappwebview.dart';
+import 'package:glider/common/constants/app_uris.dart';
 import 'package:glider/common/extensions/bloc_base_extension.dart';
 import 'package:glider_domain/glider_domain.dart';
 
@@ -22,7 +23,7 @@ class AuthCubit extends Cubit<AuthState> {
   }
 
   Future<void> login() async {
-    final userCookieUrl = WebUri('https://news.ycombinator.com');
+    final userCookieUrl = WebUri.uri(AppUris.hackerNewsUri);
     const userCookieName = 'user';
     final userCookie = await _cookieManager.getCookie(
       url: userCookieUrl,

lib/auth/view/auth_page.dart

@@ -8,6 +8,7 @@ import 'package:glider/app/container/app_container.dart';
 import 'package:glider/app/models/app_route.dart';
 import 'package:glider/auth/cubit/auth_cubit.dart';
 import 'package:glider/common/constants/app_spacing.dart';
+import 'package:glider/common/constants/app_uris.dart';
 import 'package:glider/common/extensions/uri_extension.dart';
 import 'package:glider/common/extensions/widget_list_extension.dart';
 import 'package:glider/l10n/extensions/app_localizations_extension.dart';
@@ -118,7 +119,7 @@ class _AuthPageState extends State<AuthPage> {
                   ),
                   urlRequest: URLRequest(
                     url: WebUri(
-                      Uri.https('news.ycombinator.com', 'login').toString(),
+                      AppUris.hackerNewsUri.replace(path: 'login').toString(),
                     ),
                   ),
                 ),
@@ -165,6 +166,7 @@ class _AuthBody extends StatelessWidget {
                     'github.com',
                     'Mosc/Glider/blob/master/PRIVACY.md',
                   ).tryLaunch(
+                    context,
                     useInAppBrowser: _settingsCubit.state.useInAppBrowser,
                   ),
                   child: Text(context.l10n.privacyPolicy),
@@ -174,6 +176,7 @@ class _AuthBody extends StatelessWidget {
                     'www.ycombinator.com',
                     'legal',
                   ).replace(fragment: 'privacy').tryLaunch(
+                        context,
                         useInAppBrowser: _settingsCubit.state.useInAppBrowser,
                       ),
                   child: Text(context.l10n.privacyPolicyYc),
@@ -183,6 +186,7 @@ class _AuthBody extends StatelessWidget {
                     'www.ycombinator.com',
                     'legal',
                   ).replace(fragment: 'tou').tryLaunch(
+                        context,
                         useInAppBrowser: _settingsCubit.state.useInAppBrowser,
                       ),
                   child: Text(context.l10n.termsOfUseYc),

lib/common/constants/app_uris.dart

@@ -0,0 +1,5 @@
+abstract final class AppUris {
+  static final hackerNewsUri = Uri.https('news.ycombinator.com');
+
+  static final projectUri = Uri.https('github.com', 'Mosc/Glider');
+}

lib/common/extensions/uri_extension.dart

@@ -1,7 +1,21 @@
+import 'dart:async';
+
+import 'package:flutter/material.dart';
+import 'package:glider/common/constants/app_uris.dart';
+import 'package:go_router/go_router.dart';
 import 'package:url_launcher/url_launcher.dart';
 
 extension UriExtension on Uri {
-  Future<bool> tryLaunch({String? title, required bool useInAppBrowser}) async {
+  Future<bool> tryLaunch(
+    BuildContext context, {
+    String? title,
+    required bool useInAppBrowser,
+  }) async {
+    if (authority == AppUris.hackerNewsUri.authority) {
+      unawaited(context.push(toString()));
+      return true;
+    }
+
     if (await canLaunchUrl(this)) {
       if (await supportsLaunchMode(LaunchMode.externalNonBrowserApplication)) {
         final success = await launchUrl(

lib/common/widgets/hacker_news_text.dart

@@ -90,6 +90,7 @@ class HackerNewsText extends StatelessWidget {
       onTapLink: (text, href, title) async {
         if (href != null) {
           await Uri.tryParse(href)?.tryLaunch(
+            context,
             title: title,
             useInAppBrowser: useInAppBrowser,
           );

lib/item/models/item_value.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:glider/auth/cubit/auth_cubit.dart';
+import 'package:glider/common/constants/app_uris.dart';
 import 'package:glider/common/interfaces/menu_item.dart';
 import 'package:glider/item/cubit/item_cubit.dart';
 import 'package:glider/l10n/extensions/app_localizations_extension.dart';
@@ -53,10 +54,9 @@ enum ItemValue implements MenuItem<ItemState> {
       ItemValue.title => item?.title,
       ItemValue.link => item?.url.toString(),
       ItemValue.text => item?.text,
-      ItemValue.itemLink => Uri.https(
-          'news.ycombinator.com',
-          'item',
-          <String, String>{
+      ItemValue.itemLink => AppUris.hackerNewsUri.replace(
+          path: 'item',
+          queryParameters: <String, String>{
             'id': itemCubit.itemId.toString(),
           },
         ).toString(),

lib/item/widgets/item_data_tile.dart

@@ -162,8 +162,10 @@ class ItemDataTile extends StatelessWidget {
                 AnimatedVisibility(
                   visible: style == ItemStyle.overview,
                   child: InkWell(
-                    onTap: () async =>
-                        item.url!.tryLaunch(useInAppBrowser: useInAppBrowser),
+                    onTap: () async => item.url!.tryLaunch(
+                      context,
+                      useInAppBrowser: useInAppBrowser,
+                    ),
                     // Explicitly override parent widget's long press.
                     onLongPress: () {},
                     child: _ItemFavicon(
@@ -359,7 +361,10 @@ class ItemDataTile extends StatelessWidget {
           ),
         if (item.url case final url?)
           DecoratedCard.outlined(
-            onTap: () async => url.tryLaunch(useInAppBrowser: useInAppBrowser),
+            onTap: () async => url.tryLaunch(
+              context,
+              useInAppBrowser: useInAppBrowser,
+            ),
             // Explicitly override parent widget's long press.
             onLongPress: () {},
             child: Row(

lib/settings/view/settings_page.dart

@@ -4,6 +4,7 @@ import 'package:flutter/material.dart' hide ThemeMode;
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:glider/app/models/app_route.dart';
 import 'package:glider/common/constants/app_spacing.dart';
+import 'package:glider/common/constants/app_uris.dart';
 import 'package:glider/common/extensions/uri_extension.dart';
 import 'package:glider/common/widgets/preview_card.dart';
 import 'package:glider/item/models/item_style.dart';
@@ -64,16 +65,14 @@ class _SettingsBody extends StatelessWidget {
     'Open Sans',
     'Roboto',
   ];
-  static const String _authority = 'github.com';
-  static const String _basePath = 'Mosc/Glider';
-  static final Uri _privacyPolicyUrl =
-      Uri.https(_authority, '$_basePath/blob/master/PRIVACY.md');
+  static final Uri _privacyPolicyUrl = AppUris.projectUri
+      .replace(path: '${AppUris.projectUri.path}/blob/master/PRIVACY.md');
   static const String _license = 'MIT';
-  static final Uri _licenseUrl =
-      Uri.https(_authority, '$_basePath/blob/master/LICENSE');
-  static final Uri _sourceCodeUrl = Uri.https('github.com', _basePath);
+  static final Uri _licenseUrl = AppUris.projectUri
+      .replace(path: '${AppUris.projectUri.path}/blob/master/LICENSE');
+  static final Uri _sourceCodeUrl = AppUris.projectUri;
   static final Uri _issueTrackerUrl =
-      Uri.https(_authority, '$_basePath/issues');
+      AppUris.projectUri.replace(path: '${AppUris.projectUri.path}/issues');
 
   @override
   Widget build(BuildContext context) {
@@ -332,6 +331,7 @@ class _SettingsBody extends StatelessWidget {
               title: Text(context.l10n.privacyPolicy),
               trailing: const Icon(Icons.open_in_new_outlined),
               onTap: () => _privacyPolicyUrl.tryLaunch(
+                context,
                 useInAppBrowser: state.useInAppBrowser,
               ),
             ),
@@ -340,6 +340,7 @@ class _SettingsBody extends StatelessWidget {
               subtitle: const Text(_license),
               trailing: const Icon(Icons.open_in_new_outlined),
               onTap: () => _licenseUrl.tryLaunch(
+                context,
                 useInAppBrowser: state.useInAppBrowser,
               ),
             ),
@@ -348,6 +349,7 @@ class _SettingsBody extends StatelessWidget {
               subtitle: Text(_sourceCodeUrl.toString()),
               trailing: const Icon(Icons.open_in_new_outlined),
               onTap: () => _sourceCodeUrl.tryLaunch(
+                context,
                 useInAppBrowser: state.useInAppBrowser,
               ),
             ),
@@ -356,6 +358,7 @@ class _SettingsBody extends StatelessWidget {
               subtitle: Text(_issueTrackerUrl.toString()),
               trailing: const Icon(Icons.open_in_new_outlined),
               onTap: () => _issueTrackerUrl.tryLaunch(
+                context,
                 useInAppBrowser: state.useInAppBrowser,
               ),
             ),

lib/user/models/user_value.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:glider/auth/cubit/auth_cubit.dart';
+import 'package:glider/common/constants/app_uris.dart';
 import 'package:glider/common/interfaces/menu_item.dart';
 import 'package:glider/l10n/extensions/app_localizations_extension.dart';
 import 'package:glider/settings/cubit/settings_cubit.dart';
@@ -48,10 +49,9 @@ enum UserValue implements MenuItem<UserState> {
     return switch (this) {
       UserValue.username => userCubit.username,
       UserValue.about => user?.about,
-      UserValue.userLink => Uri.https(
-          'news.ycombinator.com',
-          'user',
-          <String, String>{
+      UserValue.userLink => AppUris.hackerNewsUri.replace(
+          path: 'user',
+          queryParameters: <String, String>{
             'id': userCubit.username,
           },
         ).toString(),