Skip to content

fix some typos and grammars#1935

Open
cedricfung wants to merge 3 commits intomainfrom
strings/typos
Open

fix some typos and grammars#1935
cedricfung wants to merge 3 commits intomainfrom
strings/typos

Conversation

@cedricfung
Copy link
Contributor

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR polishes user-facing text by fixing typos/grammar, standardizing capitalization, and renaming a few localization keys so the UI can reference correctly spelled identifiers.

Changes:

  • Rename localization keys: nowAnAddminnowAnAdmin, unknowErrorunknownError, unlockWithWasscodeunlockWithPasscode, and update call sites.
  • Improve/standardize multiple English strings in intl_en.arb and regenerate localization outputs.
  • Capitalize and clarify a few device-transfer/menu labels.

Reviewed changes

Copilot reviewed 13 out of 21 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
lib/widgets/window/menus.dart Updates the debug menu label capitalization for device transfer.
lib/widgets/message/item/system_message.dart Switches to the corrected localization key nowAnAdmin.
lib/widgets/auth.dart Switches to the corrected localization key unlockWithPasscode.
lib/utils/device_transfer/device_transfer_dialog.dart Updates device transfer UI strings (capitalization/wording).
lib/ui/landing/landing.dart Switches to the corrected localization key unknownError.
lib/l10n/intl_zh.arb Renames corrected keys (nowAnAdmin, unknownError, unlockWithPasscode).
lib/l10n/intl_zh-TW.arb Renames corrected keys (nowAnAdmin, unknownError, unlockWithPasscode).
lib/l10n/intl_zh-HK.arb Renames corrected keys (nowAnAdmin, unknownError, unlockWithPasscode).
lib/l10n/intl_ru.arb Renames corrected key (nowAnAdmin).
lib/l10n/intl_ja.arb Renames corrected key (nowAnAdmin).
lib/l10n/intl_es.arb Renames corrected key (nowAnAdmin).
lib/l10n/intl_en.arb Fixes many English strings and renames corrected keys.
lib/generated/l10n.dart Regenerated localization accessors; includes renamed keys and updated messages.
lib/generated/intl/messages_zh.dart Regenerated message map for renamed keys.
lib/generated/intl/messages_zh-TW.dart Regenerated message map for renamed keys.
lib/generated/intl/messages_zh-HK.dart Regenerated message map for renamed keys.
lib/generated/intl/messages_ru.dart Regenerated message map for renamed keys.
lib/generated/intl/messages_ja.dart Regenerated message map for renamed keys.
lib/generated/intl/messages_es.dart Regenerated message map for renamed keys.
lib/generated/intl/messages_en.dart Regenerated message map for updated English strings and renamed keys.
lib/app.dart Switches to the corrected localization key unknownError.
Comments suppressed due to low confidence (1)

lib/widgets/window/menus.dart:280

  • This menu item label is hard-coded while the surrounding menu labels use context.l10n.*. To keep the window menu translatable/consistent, please add a localization key for this label and use it here (even if it’s currently behind kDebugMode).
                    PlatformMenuItem(
                      label: 'Chat Backup and Restore',
                      onSelected: signed
                          ? () {
                              showDeviceTransferDialog(context);
                            }

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines 109 to 114
CellGroup(
padding: const EdgeInsets.symmetric(horizontal: 20),
child: CellItem(
title: const Text('sync from other device'),
title: const Text('Sync from Another Device'),
onTap: () {
context.read<_NavigatorCubit>().push(
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants