What the developer wrote
Authored by ww3512687
fix nabox wallet connect wallet
What changed, and why it matters
This commit changes which cryptocurrency filters are shown for the Nabox wallet in the wallet connection screen. It expands the filter categories from just 'Other' to include Bitcoin, Ethereum, Solana, and Other. This is a UI/UX fix with no apparent security relevance.
No security action required. This is a routine UI categorization fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch modifies a single entry in the wallet list array for the Nabox wallet, changing the filter bitmask from WALLET_FILTER_OTHER to a combination of WALLET_FILTER_BTC | WALLET_FILTER_ETH | WALLET_FILTER_SOL | WALLET_FILTER_OTHER. This affects how the wallet is categorized/filtered in the GUI but does not alter cryptographic, authentication, or communication logic.
Changed components
src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.cInspect captured patch +1 / −1
diff --git a/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c b/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c
index b1aaf2f..c595c52 100644
--- a/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c
+++ b/src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.c
@@ -223,7 +223,7 @@ WalletListItem_t g_walletListArray[] = {
{WALLET_LIST_BTC_WALLET, &coinBtc, "Bitcoin Wallets", g_btcWalletCoinArray, 4, true, WALLET_FILTER_BTC},
{WALLET_LIST_TONKEEPER, &walletTonkeeper, "Tonkeeper", g_tonKeeperCoinArray, 1, false, WALLET_FILTER_OTHER},
{WALLET_LIST_RABBY, &walletRabby, "Rabby", g_ethWalletCoinArray, 4, true, WALLET_FILTER_ETH},
- {WALLET_LIST_NABOX, &walletNabox, "Nabox", g_naboxWalletCoinArray, 7, true, WALLET_FILTER_OTHER},
+ {WALLET_LIST_NABOX, &walletNabox, "Nabox", g_naboxWalletCoinArray, 7, true, WALLET_FILTER_BTC | WALLET_FILTER_ETH | WALLET_FILTER_SOL | WALLET_FILTER_OTHER},
{WALLET_LIST_BITGET, &walletBitget, "Bitget Wallet", g_bitgetWalletCoinArray, 3, true, WALLET_FILTER_BTC | WALLET_FILTER_ETH | WALLET_FILTER_OTHER},
{WALLET_LIST_ETERNL, &walletEternl, "Eternl", g_adaCoinArray, 1, true, WALLET_FILTER_ADA},
{WALLET_LIST_VESPR, &walletVespr, "Vespr", g_adaCoinArray, 1, true, WALLET_FILTER_ADA},
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.