fix missing sol transactions list on desktop
What the developer wrote
Authored by julian
fix missing sol transactions list on desktop
What changed, and why it matters
This commit fixes a user-interface bug on the desktop version of Stack Wallet where the 'Transactions' tab was missing for Solana (SOL) wallets. The change simply adds Solana alongside Ethereum so that the transactions list is shown. There is no indication this is a security issue; it is a UI/UX fix.
No security action required. Treat as a routine UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In my_wallet.dart, two conditional checks that previously gated the ‘Transactions’ tab and its corresponding view to Ethereum (isEth) now also include Solana (isSolana). This is a two-line logic expansion with no changes to transaction handling, cryptography, networking, or data validation.
Changed components
lib/pages_desktop_specific/my_stack_view/wallet_view/sub_widgets/my_wallet.dartInspect captured patch +2 / −2
diff --git a/lib/pages_desktop_specific/my_stack_view/wallet_view/sub_widgets/my_wallet.dart b/lib/pages_desktop_specific/my_stack_view/wallet_view/sub_widgets/my_wallet.dart
index 7be1146..dad37c3 100644
--- a/lib/pages_desktop_specific/my_stack_view/wallet_view/sub_widgets/my_wallet.dart
+++ b/lib/pages_desktop_specific/my_stack_view/wallet_view/sub_widgets/my_wallet.dart
@@ -67,7 +67,7 @@ class _MyWalletState extends ConsumerState<MyWallet> {
titles.add("Finalize");
}
- if (isEth && widget.contractAddress == null) {
+ if ((isEth || isSolana) && widget.contractAddress == null) {
titles.add("Transactions");
}
@@ -195,7 +195,7 @@ class _MyWalletState extends ConsumerState<MyWallet> {
child: EpicFinalizeView(walletId: widget.walletId),
),
- if (isEth && widget.contractAddress == null)
+ if ((isEth || isSolana) && widget.contractAddress == null)
Padding(
padding: const EdgeInsets.only(top: 8.0),
child: ConstrainedBox(
Why this scored 19/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.