login: Add translations

sirpengi · sirpengi · commit 742b92b27c48 · 2023-10-02T19:17:44.000+01:00
diff --git a/assets/l10n/app_en.arb b/assets/l10n/app_en.arb
@@ -59,6 +59,25 @@
   "@errorCopyingFailed": {
     "description": "Error message when copying the text of a message to the users system clipboard failed."
   },
+  "errorLoginInvalidInputTitle": "Invalid input",
+  "@errorLoginInvalidInputTitle": {
+    "description": "Error title for login when input is invalid."
+  },
+  "errorLoginFailed": "Login failed",
+  "@errorLoginFailed": {
+    "description": "Error title for login when signing into a Zulip server fails."
+  },
+  "errorLoginCouldNotConnect": "Failed to connect to server:\n{url}",
+  "@errorLoginCouldNotConnect": {
+    "description": "Error message when the app could not connect to the server.",
+    "placeholders": {
+      "url": {"type": "String", "example": "http://example.com/"}
+    }
+  },
+  "errorLoginCouldNotConnectTitle": "Could not connect",
+  "@errorLoginCouldNotConnectTitle": {
+    "description": "Error title when the app could not connect to the server."
+  },
   "errorMessageDoesNotSeemToExist": "That message does not seem to exist.",
   "@errorMessageDoesNotSeemToExist": {
     "description": "Error message when loading a message that does not exist."
@@ -67,6 +86,13 @@
   "@errorQuotationFailed": {
     "description": "Error message when quoting a message failed."
   },
+  "errorServerMessage": "The server said:\n\n{message}",
+  "@errorServerMessage": {
+    "description": "Error message that quotes an error from the server.",
+    "placeholders": {
+      "message": {"type": "String", "example": "Invalid format"}
+    }
+  },
   "successLinkCopied": "Link copied",
   "@successLinkCopied": {
     "description": "Success message after copy link action completed."
@@ -107,6 +133,50 @@
   "@lightboxCopyLinkTooltip": {
     "description": "Tooltip in lightbox for the copy link action."
   },
+  "loginPageTitle": "Log in",
+  "@loginPageTitle": {
+    "description": "Page title for login page."
+  },
+  "loginFormSubmitLabel": "Log in",
+  "@loginFormSubmitLabel": {
+    "description": "Button text to submit login credentials."
+  },
+  "loginAddAnAccount": "Add an account",
+  "@loginAddAnAccount": {
+    "description": "Page title for screen to add a Zulip account."
+  },
+  "loginServerUrlInputLabel": "Your Zulip server URL",
+  "@loginServerUrlInputLabel": {
+    "description": "Input label in login page for Zulip server URL entry."
+  },
+  "loginHidePassword": "Hide password",
+  "@loginHidePassword": {
+    "description": "Icon label for button to hide password in input form."
+  },
+  "loginEmailLabel": "Email address",
+  "@loginEmailLabel": {
+    "description": "Label for input when an email is required to login."
+  },
+  "loginErrorMissingEmail": "Please enter your email.",
+  "@loginErrorMissingEmail": {
+    "description": "Error message when an empty email was provided."
+  },
+  "loginPasswordLabel": "Password",
+  "@loginPasswordLabel": {
+    "description": "Label for input for password field."
+  },
+  "loginErrorMissingPassword": "Please enter your password.",
+  "@loginErrorMissingPassword": {
+    "description": "Error message when an empty password was provided."
+  },
+  "loginUsernameLabel": "Username",
+  "@loginUsernameLabel": {
+    "description": "Label for input when a username is required to login."
+  },
+  "loginErrorMissingUsername": "Please enter your username.",
+  "@loginErrorMissingUsername": {
+    "description": "Error message when an empty username was provided."
+  },
   "topicValidationErrorTooLong": "Topic length shouldn't be greater than 60 characters.",
   "@topicValidationErrorTooLong": {
     "description": "Topic validation error when topic is too long."
diff --git a/lib/widgets/login.dart b/lib/widgets/login.dart
@@ -141,7 +141,7 @@ class _AddAccountPageState extends State<AddAccountPage> {
     final error = _parseResult.error;
     if (error != null) {
       showErrorDialog(context: context,
-        title: 'Invalid input',
+        title: zulipLocalizations.errorLoginInvalidInputTitle,
         message: error.message(zulipLocalizations));
       return;
     }
@@ -161,7 +161,8 @@ class _AddAccountPageState extends State<AddAccountPage> {
         // TODO(#105) give more helpful feedback; see `fetchServerSettings`
         //   in zulip-mobile's src/message/fetchActions.js.
         showErrorDialog(context: context,
-          title: 'Could not connect', message: 'Failed to connect to server:\n$url');
+          title: zulipLocalizations.errorLoginCouldNotConnectTitle,
+          message: zulipLocalizations.errorLoginCouldNotConnect(url.toString()));
         return;
       }
       // https://github.com/dart-lang/linter/issues/4007
@@ -190,7 +191,7 @@ class _AddAccountPageState extends State<AddAccountPage> {
       : error.message(zulipLocalizations);
 
     return Scaffold(
-      appBar: AppBar(title: const Text('Add an account'),
+      appBar: AppBar(title: Text(zulipLocalizations.loginAddAnAccount),
         bottom: _inProgress
           ? const PreferredSize(preferredSize: Size.fromHeight(4),
               child: LinearProgressIndicator(minHeight: 4)) // 4 restates default
@@ -215,7 +216,7 @@ class _AddAccountPageState extends State<AddAccountPage> {
                   // …but leave out unfocusing the input in case more editing is needed.
                 },
                 decoration: InputDecoration(
-                  labelText: 'Your Zulip server URL',
+                  labelText: zulipLocalizations.loginServerUrlInputLabel,
                   errorText: errorText,
                   helperText: kLayoutPinningHelperText,
                   hintText: 'your-org.zulipchat.com')),
@@ -224,7 +225,7 @@ class _AddAccountPageState extends State<AddAccountPage> {
                 onPressed: !_inProgress && errorText == null
                   ? () => _onSubmitted(context)
                   : null,
-                child: const Text('Continue')),
+                child: Text(zulipLocalizations.dialogContinue)),
             ])))));
   }
 }
@@ -293,10 +294,13 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
         // TODO(#105) give more helpful feedback. The RN app is
         //   unhelpful here; we should at least recognize invalid auth errors, and
         //   errors for deactivated user or realm (see zulip-mobile#4571).
+        final zulipLocalizations = ZulipLocalizations.of(context);
         final message = (e is ZulipApiException)
-          ? 'The server said:\n\n${e.message}'
+          ? zulipLocalizations.errorServerMessage(e.message)
           : e.message;
-        showErrorDialog(context: context, title: 'Login failed', message: message);
+        showErrorDialog(context: context,
+          title: zulipLocalizations.errorLoginFailed,
+          message: message);
         return;
       }
 
@@ -339,6 +343,7 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
   @override
   Widget build(BuildContext context) {
     assert(!PerAccountStoreWidget.debugExistsOf(context));
+    final zulipLocalizations = ZulipLocalizations.of(context);
     final requireEmailFormatUsernames = widget.serverSettings.requireEmailFormatUsernames;
 
     final usernameField = TextFormField(
@@ -354,8 +359,8 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
       validator: (value) {
         if (value == null || value.trim().isEmpty) {
           return requireEmailFormatUsernames
-            ? 'Please enter your email.'
-            : 'Please enter your username.';
+            ? zulipLocalizations.loginErrorMissingEmail
+            : zulipLocalizations.loginErrorMissingUsername;
         }
         if (requireEmailFormatUsernames) {
           // TODO(#106): validate is in the shape of an email
@@ -364,7 +369,9 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
       },
       textInputAction: TextInputAction.next,
       decoration: InputDecoration(
-        labelText: requireEmailFormatUsernames ? 'Email address' : 'Username',
+        labelText: requireEmailFormatUsernames
+          ? zulipLocalizations.loginEmailLabel
+          : zulipLocalizations.loginUsernameLabel,
         helperText: kLayoutPinningHelperText,
       ));
 
@@ -376,14 +383,14 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
       autovalidateMode: AutovalidateMode.onUserInteraction,
       validator: (value) {
         if (value == null || value.isEmpty) {
-          return 'Please enter your password.';
+          return zulipLocalizations.loginErrorMissingPassword;
         }
         return null;
       },
       textInputAction: TextInputAction.go,
       onFieldSubmitted: (value) => _submit(),
       decoration: InputDecoration(
-        labelText: 'Password',
+        labelText: zulipLocalizations.loginPasswordLabel,
         helperText: kLayoutPinningHelperText,
         // TODO(material-3): Simplify away `Semantics` by using IconButton's
         //   M3-only params `isSelected` / `selectedIcon`, after fixing
@@ -393,14 +400,14 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
         //   [ButtonStyleButton].)
         suffixIcon: Semantics(toggled: _obscurePassword,
           child: IconButton(
-            tooltip: 'Hide password',
+            tooltip: zulipLocalizations.loginHidePassword,
             onPressed: _handlePasswordVisibilityPress,
             icon: _obscurePassword
               ? const Icon(Icons.visibility_off)
               : const Icon(Icons.visibility)))));
 
     return Scaffold(
-      appBar: AppBar(title: const Text('Log in'),
+      appBar: AppBar(title: Text(zulipLocalizations.loginPageTitle),
         bottom: _inProgress
           ? const PreferredSize(preferredSize: Size.fromHeight(4),
               child: LinearProgressIndicator(minHeight: 4)) // 4 restates default
@@ -420,7 +427,7 @@ class _PasswordLoginPageState extends State<PasswordLoginPage> {
                   const SizedBox(height: 8),
                   ElevatedButton(
                     onPressed: _inProgress ? null : _submit,
-                    child: const Text('Log in')),
+                    child: Text(zulipLocalizations.loginFormSubmitLabel)),
                 ])))))));
   }
 }
