db: Support setting browser preference

Signed-off-by: Zixuan James Li <[email protected]>

Author: PIG208

Diff for: assets/l10n/app_en.arb

@@ -23,6 +23,10 @@
   "@settingsPageTitle": {
     "description": "Title for the settings page."
   },
+  "settingsUseExternal": "Use external browser",
+  "@settingsUseExternal": {
+    "description": "Whether to open links with external browser or not"
+  },
   "switchAccountButton": "Switch account",
   "@switchAccountButton": {
     "description": "Label for main-menu button leading to the choose-account page."

Diff for: lib/generated/l10n/zulip_localizations.dart

@@ -141,6 +141,12 @@ abstract class ZulipLocalizations {
   /// **'Settings'**
   String get settingsPageTitle;
 
+  /// Whether to open links with external browser or not
+  ///
+  /// In en, this message translates to:
+  /// **'Use external browser'**
+  String get settingsUseExternal;
+
   /// Label for main-menu button leading to the choose-account page.
   ///
   /// In en, this message translates to:

Diff for: lib/generated/l10n/zulip_localizations_ar.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Switch account';
 

Diff for: lib/generated/l10n/zulip_localizations_en.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Switch account';
 

Diff for: lib/generated/l10n/zulip_localizations_ja.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Switch account';
 

Diff for: lib/generated/l10n/zulip_localizations_nb.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Switch account';
 

Diff for: lib/generated/l10n/zulip_localizations_pl.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Przełącz konto';
 

Diff for: lib/generated/l10n/zulip_localizations_ru.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Сменить учетную запись';
 

Diff for: lib/generated/l10n/zulip_localizations_sk.dart

@@ -26,6 +26,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
   @override
   String get settingsPageTitle => 'Settings';
 
+  @override
+  String get settingsUseExternal => 'Use external browser';
+
   @override
   String get switchAccountButton => 'Zmeniť účet';
 

Diff for: lib/model/database.dart

@@ -53,6 +53,9 @@ class Accounts extends Table {
 class GlobalSettings extends Table {
   Column<String> get themeSetting => textEnum<ThemeSetting>()
     .withDefault(const Variable('unset'))();
+
+  Column<String> get browserPreference => textEnum<BrowserPreference>()
+    .nullable()();
 }
 
 class UriConverter extends TypeConverter<Uri, String> {
@@ -73,7 +76,7 @@ class AppDatabase extends _$AppDatabase {
   //  * Write a migration in `onUpgrade` below.
   //  * Write tests.
   @override
-  int get schemaVersion => 3; // See note.
+  int get schemaVersion => 4; // See note.
 
   Future<void> _resetDatabase(Migrator m) async {
     // This should only ever happen in dev.  As a dev convenience,
@@ -117,6 +120,10 @@ class AppDatabase extends _$AppDatabase {
             from2To3: (m, schema) async {
               await m.createTable(schema.globalSettings);
             },
+            from3To4: (m, schema) async {
+              await m.addColumn(
+                schema.globalSettings, schema.globalSettings.browserPreference);
+            },
           ));
       });
   }