feat(shopinbit): add ShopInBit More menu item on mobile
What the developer wrote
Authored by sneurlax
feat(shopinbit): add ShopInBit More menu item on mobile
What changed, and why it matters
This commit simply adds a new 'Services' menu item to the bottom navigation bar in the mobile wallet view. It lets users open an existing screen called ServicesView. There is no security-relevant change in the code shown.
No security action needed. Review the target ServicesView separately if assessing the security of that feature.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds one import and one WalletNavigationBarItemData entry in lib/pages/wallet_view/wallet_view.dart. The new item is conditionally shown when the wallet is not view-only and navigates to ServicesView.routeName. No logic changes to cryptography, networking, permissions, input handling, or data storage are present.
Changed components
lib/pages/wallet_view/wallet_view.dartInspect captured patch +18 / −0
diff --git a/lib/pages/wallet_view/wallet_view.dart b/lib/pages/wallet_view/wallet_view.dart
index 74a129e..24e2e7b 100644
--- a/lib/pages/wallet_view/wallet_view.dart
+++ b/lib/pages/wallet_view/wallet_view.dart
@@ -109,6 +109,7 @@ import '../settings_views/wallet_settings_view/wallet_network_settings_view/wall
import '../settings_views/wallet_settings_view/wallet_settings_view.dart';
import '../signing/signing_view.dart';
import '../spark_names/spark_names_home_view.dart';
+import '../more_view/services_view.dart';
import '../token_view/my_tokens_view.dart';
import 'sub_widgets/transactions_list.dart';
import 'sub_widgets/wallet_summary.dart';
@@ -1343,6 +1344,23 @@ class _WalletViewState extends ConsumerState<WalletView> {
);
},
),
+ if (!viewOnly)
+ WalletNavigationBarItemData(
+ label: "Services",
+ icon: SvgPicture.asset(
+ Assets.svg.circleSliders,
+ height: 20,
+ width: 20,
+ color: Theme.of(
+ context,
+ ).extension<StackColors>()!.bottomNavIconIcon,
+ ),
+ onTap: () {
+ Navigator.of(context).pushNamed(
+ ServicesView.routeName,
+ );
+ },
+ ),
],
),
),
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.