From 89767a91217e15e2999354fb3e3ece772bcf5022 Mon Sep 17 00:00:00 2001
From: Dominik Toton <dominik.toton@iohk.io>
Date: Tue, 24 Sep 2024 21:23:17 +0200
Subject: [PATCH] refactor(cat-voices): cleanup

---
 .../link_wallet/intro/link_wallet_intro_dialog.dart       | 8 ++++----
 .../registration/link_wallet/link_wallet_dialog.dart      | 4 ++--
 .../buttons/specialized/voices_learn_more_button.dart     | 1 +
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/catalyst_voices/lib/pages/registration/link_wallet/intro/link_wallet_intro_dialog.dart b/catalyst_voices/lib/pages/registration/link_wallet/intro/link_wallet_intro_dialog.dart
index c8a2d85a4c..a9f1149d75 100644
--- a/catalyst_voices/lib/pages/registration/link_wallet/intro/link_wallet_intro_dialog.dart
+++ b/catalyst_voices/lib/pages/registration/link_wallet/intro/link_wallet_intro_dialog.dart
@@ -5,13 +5,13 @@ import 'package:catalyst_voices_assets/catalyst_voices_assets.dart';
 import 'package:catalyst_voices_localization/catalyst_voices_localization.dart';
 import 'package:flutter/material.dart';
 
-// TODO(dtscalac): add content for the screen
+/// The initial screen for the link wallet flow during registration.
 class LinkWalletIntroDialog extends StatelessWidget {
-  final VoidCallback onChooseCardanoWallet;
+  final VoidCallback onSelectWallet;
 
   const LinkWalletIntroDialog({
     super.key,
-    required this.onChooseCardanoWallet,
+    required this.onSelectWallet,
   });
 
   @override
@@ -53,7 +53,7 @@ class LinkWalletIntroDialog extends StatelessWidget {
           const Spacer(),
           VoicesFilledButton(
             leading: VoicesAssets.icons.wallet.buildIcon(),
-            onTap: onChooseCardanoWallet,
+            onTap: onSelectWallet,
             child: Text(context.l10n.chooseCardanoWallet),
           ),
         ],
diff --git a/catalyst_voices/lib/pages/registration/link_wallet/link_wallet_dialog.dart b/catalyst_voices/lib/pages/registration/link_wallet/link_wallet_dialog.dart
index 9887c31725..0fe4456657 100644
--- a/catalyst_voices/lib/pages/registration/link_wallet/link_wallet_dialog.dart
+++ b/catalyst_voices/lib/pages/registration/link_wallet/link_wallet_dialog.dart
@@ -30,7 +30,7 @@ class _LinkWalletDialogState extends State<LinkWalletDialog> {
   Widget build(BuildContext context) {
     return switch (_stage) {
       LinkWalletStage.intro => LinkWalletIntroDialog(
-          onChooseCardanoWallet: _onChooseCardanoWallet,
+          onSelectWallet: _onSelectWallet,
         ),
       LinkWalletStage.selectWallet => SelectWalletDialog(
           onSelectedWallet: _onSelectedWallet,
@@ -38,7 +38,7 @@ class _LinkWalletDialogState extends State<LinkWalletDialog> {
     };
   }
 
-  void _onChooseCardanoWallet() {
+  void _onSelectWallet() {
     setState(() {
       _stage = LinkWalletStage.selectWallet;
     });
diff --git a/catalyst_voices/lib/widgets/buttons/specialized/voices_learn_more_button.dart b/catalyst_voices/lib/widgets/buttons/specialized/voices_learn_more_button.dart
index b40819e7f9..93386dc013 100644
--- a/catalyst_voices/lib/widgets/buttons/specialized/voices_learn_more_button.dart
+++ b/catalyst_voices/lib/widgets/buttons/specialized/voices_learn_more_button.dart
@@ -3,6 +3,7 @@ import 'package:catalyst_voices_assets/catalyst_voices_assets.dart';
 import 'package:catalyst_voices_localization/catalyst_voices_localization.dart';
 import 'package:flutter/material.dart';
 
+/// A "Learn More" button that redirects usually to an external content.
 class VoicesLearnMoreButton extends StatelessWidget {
   final VoidCallback onTap;