feat(backup): make create-backup a button and tag the vault as Tor-using

- start-backup becomes an outlined button between the hero and the
  settings list instead of a menu row: creating a backup is an action,
  not a setting, and it should read as one even when a backup exists.
  It keeps the translated label (27 locales) and is still suppressed in
  the zero-backup state, where the hero already offers it.
- BackupOptionCard takes a list of tags so a card can carry more than
  one; the encrypted vault card now also shows 'Uses Tor', since
  reaching the key server over Tor is a property worth knowing before
  choosing that option.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: FrancisPouliot

lib/core/widgets/cards/backup_option_card.dart

@@ -8,15 +8,18 @@ class BackupOptionCard extends StatelessWidget {
   final Widget icon;
   final String title;
   final String description;
-  final String? tag;
+
+  /// Zero or more short labels shown under the description. A card may carry
+  /// several (e.g. how easy an option is AND how it reaches the network).
+  final List<String> tags;
   final VoidCallback onTap;
 
   const BackupOptionCard({
     super.key,
     required this.icon,
     required this.title,
     required this.description,
-    this.tag,
+    this.tags = const [],
     required this.onTap,
   });
 
@@ -60,7 +63,16 @@ class BackupOptionCard extends StatelessWidget {
                           maxLines: 3,
                         ),
                         const Gap(10),
-                        if (tag != null) OptionsTag(text: tag!),
+                        // Tags wrap so a card can carry more than one without
+                        // overflowing on narrow screens.
+                        if (tags.isNotEmpty)
+                          Wrap(
+                            spacing: 8,
+                            runSpacing: 8,
+                            children: [
+                              for (final tag in tags) OptionsTag(text: tag),
+                            ],
+                          ),
                       ],
                     ),
                   ),

lib/features/backup_settings/ui/screens/backup_options_screen.dart

@@ -59,7 +59,10 @@ class _BackupOptionsScreenState extends State<BackupOptionsScreen> {
                 ),
                 title: context.loc.backupWalletEncryptedVaultTitle,
                 description: context.loc.backupWalletEncryptedVaultDescription,
-                tag: context.loc.backupWalletEncryptedVaultTag,
+                tags: [
+                  context.loc.backupWalletEncryptedVaultTag,
+                  context.loc.backupWalletEncryptedVaultUsesTorTag,
+                ],
                 onTap: () => context.pushNamed(
                   RecoverBullRoute.recoverbullFlows.name,
                   extra: RecoverBullFlowsExtra(
@@ -82,7 +85,7 @@ class _BackupOptionsScreenState extends State<BackupOptionsScreen> {
                 ),
                 title: context.loc.backupWalletPhysicalBackupTitle,
                 description: context.loc.backupWalletPhysicalBackupDescription,
-                tag: context.loc.backupWalletPhysicalBackupTag,
+                tags: [context.loc.backupWalletPhysicalBackupTag],
                 onTap: () {
                   context.pushNamed(
                     TestWalletBackupRoute.testPhysicalBackupFlow.name,

lib/features/backup_settings/ui/screens/backup_settings_screen.dart

@@ -85,6 +85,14 @@ class _Screen extends StatelessWidget {
                         ),
                       ),
                       if (hero != null) ...[const Gap(32), hero],
+                      // The create-backup action sits between the hero and the
+                      // settings list: it is an action, not a setting, and it
+                      // stays available in every state except the zero-backup
+                      // one, where the hero already offers it.
+                      if (!_heroOffersStartBackup(state)) ...[
+                        const Gap(24),
+                        const _StartBackupButton(),
+                      ],
                       const Gap(24),
                       ..._menuRows(state),
                     ],
@@ -144,12 +152,8 @@ class _Screen extends StatelessWidget {
   /// The settings rows. Insertion point for the fork's metadata backup menu
   /// row: add one [SettingsEntryItem] to this list.
   List<Widget> _menuRows(BackupSettingsState state) => [
-    // Availability is not encouragement. The hero never asks for another
-    // backup once a physical one is fresh, but adding one deliberately has to
-    // stay possible from here in every state — EXCEPT the one state where the
-    // hero is already offering this exact action, where a second identical
-    // entry a few pixels below it is just noise.
-    if (!_heroOffersStartBackup(state)) const _StartBackupButton(),
+    // Start-backup is NOT here: it is an action, rendered as a button above
+    // this list (see the body), not a settings row.
     if (state.lastEncryptedBackup != null) const _ViewVaultKeyButton(),
     if (state.lastEncryptedBackup != null || state.lastPhysicalBackup != null)
       const _TestBackupButton(),
@@ -341,18 +345,31 @@ class _TestBackupHero extends StatelessWidget {
   }
 }
 
+/// Creating a backup is an ACTION, not a setting, so it is a button rather
+/// than a menu row — obvious at a glance even when a backup already exists.
+/// Outlined rather than filled: it must not compete with a hero's own CTA when
+/// one is present, and it is an offer, never a warning.
 class _StartBackupButton extends StatelessWidget {
   const _StartBackupButton();
 
   @override
   Widget build(BuildContext context) {
-    return SettingsEntryItem(
-      icon: Icons.save_as,
-      iconColor: context.appColors.primary,
-      title: context.loc.backupSettingsStartBackup,
-      onTap: () => context.pushNamed(
-        BackupSettingsSubroute.backupOptions.name,
-        extra: BackupSettingsFlow.backup,
+    return Padding(
+      padding: const EdgeInsets.symmetric(horizontal: 16),
+      child: BBButton.big(
+        // The translated row label (27 locales), not the hero's EN-only CTA
+        // string: a button in the user's own language beats an English one.
+        label: context.loc.backupSettingsStartBackup,
+        iconData: Icons.save_as,
+        iconFirst: true,
+        onPressed: () => context.pushNamed(
+          BackupSettingsSubroute.backupOptions.name,
+          extra: BackupSettingsFlow.backup,
+        ),
+        outlined: true,
+        bgColor: context.appColors.transparent,
+        textColor: context.appColors.onSurface,
+        borderColor: context.appColors.outline,
       ),
     );
   }

localization/app_en.arb

@@ -15069,5 +15069,9 @@
   "recoverbullServerCustomWarning": "You are about to use a custom Recoverbull server. Make sure you trust this server.",
   "@recoverbullServerCustomWarning": {
     "description": "Warning shown before saving a custom key server URL in the encrypted vault settings"
+  },
+  "backupWalletEncryptedVaultUsesTorTag": "Uses Tor",
+  "@backupWalletEncryptedVaultUsesTorTag": {
+    "description": "Tag on the encrypted vault option card indicating the key server is reached over Tor"
   }
 }