settings: Add browser preference setting ui

The translation for the label is taken from zulip-mobile. The user
facing string refers to "in in-app browser" when the actual value is
`UrlLaunchMode.platformDefault`. This is consistent with the behavior
the user sees when the link is a HTTP URL on Android and iOS, which
should cover the majority of use cases of this setting (and probably
what the user would expect).

The UI currently does not have a design, and is made to be as simple as
possible to implement for now.

Fixes: #1228

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

Author: PIG208

Diff for: assets/l10n/app_en.arb

@@ -811,6 +811,10 @@
   "@themeSettingSystem": {
     "description": "Label for system theme setting."
   },
+  "openLinksWithInAppBrowser": "Open links with in-app browser",
+  "@openLinksWithInAppBrowser": {
+    "description": "Label for toggling setting to open links with in-app browser"
+  },
   "pollWidgetQuestionMissing": "No question.",
   "@pollWidgetQuestionMissing": {
     "description": "Text to display for a poll when the question is missing"

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

@@ -1185,6 +1185,12 @@ abstract class ZulipLocalizations {
   /// **'System'**
   String get themeSettingSystem;
 
+  /// Label for toggling setting to open links with in-app browser
+  ///
+  /// In en, this message translates to:
+  /// **'Open links with in-app browser'**
+  String get openLinksWithInAppBrowser;
+
   /// Text to display for a poll when the question is missing
   ///
   /// In en, this message translates to:

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'No question.';
 

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'No question.';
 

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'No question.';
 

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'No question.';
 

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'Brak pytania.';
 

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'Нет вопроса.';
 

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

@@ -634,6 +634,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
   @override
   String get themeSettingSystem => 'System';
 
+  @override
+  String get openLinksWithInAppBrowser => 'Open links with in-app browser';
+
   @override
   String get pollWidgetQuestionMissing => 'Bez otázky.';
 

Diff for: lib/widgets/settings.dart

@@ -24,6 +24,7 @@ class SettingsPage extends StatelessWidget {
         title: Text(zulipLocalizations.settingsPageTitle)),
       body: Column(children: [
         const _ThemeSetting(),
+        const _BrowserPreferenceSetting(),
       ]));
   }
 }
@@ -54,3 +55,26 @@ class _ThemeSetting extends StatelessWidget {
       ]);
   }
 }
+
+class _BrowserPreferenceSetting extends StatelessWidget {
+  const _BrowserPreferenceSetting();
+
+  void _handleChange(BuildContext context, bool newOpenLinksWithInAppBrowser) {
+    GlobalStoreWidget.of(context).updateGlobalSettings(
+      GlobalSettingsCompanion(browserPreference: Value(
+        newOpenLinksWithInAppBrowser ? BrowserPreference.inApp
+                                     : BrowserPreference.external)));
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    final zulipLocalizations = ZulipLocalizations.of(context);
+    final openLinksWithInAppBrowser =
+      GlobalStoreWidget.of(context).globalSettings.effectiveBrowserPreference
+      == BrowserPreference.inApp;
+    return SwitchListTile.adaptive(
+      title: Text(zulipLocalizations.openLinksWithInAppBrowser),
+      value: openLinksWithInAppBrowser,
+      onChanged: (newValue) => _handleChange(context, newValue));
+  }
+}

Diff for: test/flutter_checks.dart

@@ -176,6 +176,10 @@ extension RadioListTileChecks<T> on Subject<RadioListTile<T>> {
   Subject<bool> get checked => has((x) => x.checked, 'checked');
 }
 
+extension SwitchListTileChecks<T> on Subject<SwitchListTile> {
+  Subject<bool> get value => has((x) => x.value, 'value');
+}
+
 extension ThemeDataChecks on Subject<ThemeData> {
   Subject<Brightness> get brightness => has((x) => x.brightness, 'brightness');
 }

Diff for: test/widgets/settings_test.dart

@@ -83,4 +83,49 @@ void main() {
       debugBrightnessOverride = null;
     });
   });
+
+  group('BrowserPreference', () {
+    Finder useInAppBrowserSwitchFinder = find.ancestor(
+      of: find.text('Open links with in-app browser'),
+      matching: find.byType(SwitchListTile));
+
+    void checkSwitchAndGlobalSettings(WidgetTester tester, {
+      required bool checked,
+      required BrowserPreference? expectedBrowserPreference,
+    }) {
+      check(tester.widget<SwitchListTile>(useInAppBrowserSwitchFinder))
+        .value.equals(checked);
+      check(testBinding.globalStore)
+        .globalSettings.browserPreference.equals(expectedBrowserPreference);
+    }
+
+    testWidgets('smoke', (tester) async {
+      await testBinding.globalStore.updateGlobalSettings(
+        eg.globalSettings(
+          browserPreference: BrowserPreference.external).toCompanion(false));
+      await prepare(tester);
+      checkSwitchAndGlobalSettings(tester,
+        checked: false, expectedBrowserPreference: BrowserPreference.external);
+
+      await tester.tap(useInAppBrowserSwitchFinder);
+      await tester.pump();
+      checkSwitchAndGlobalSettings(tester,
+        checked: true, expectedBrowserPreference: BrowserPreference.inApp);
+    });
+
+    testWidgets('use platform-specific default browser preference', (tester) async {
+      await prepare(tester);
+      bool useInAppBrowserExpected = defaultTargetPlatform == TargetPlatform.android;
+      checkSwitchAndGlobalSettings(tester,
+        checked: useInAppBrowserExpected, expectedBrowserPreference: null);
+
+      await tester.tap(useInAppBrowserSwitchFinder);
+      await tester.pump();
+      useInAppBrowserExpected = !useInAppBrowserExpected;
+      checkSwitchAndGlobalSettings(tester,
+        checked: useInAppBrowserExpected,
+        expectedBrowserPreference: useInAppBrowserExpected
+          ? BrowserPreference.inApp : BrowserPreference.external);
+    }, variant: TargetPlatformVariant({TargetPlatform.android, TargetPlatform.iOS}));
+  });
 }