What the developer wrote
Authored by soralit
fix: zec logo issue
What changed, and why it matters
This commit fixes a display issue by adding the Zcash (ZEC) coin logo to the list of icons shown for the Keystone wallet connection screen. It also removes a few leftover blank lines and an outdated 'todo' comment. There is no security-relevant change in this patch.
No security action needed. Treat as a routine UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies a single UI source file. The only functional change is appending &coinZec to the g_keystoneWalletCoinArray array so the Zcash logo appears alongside other coin logos in the wallet-connect UI. The remaining changes are cosmetic: deletion of empty lines and removal of a stale // todo add ao comment (the AO coin icon is already present). No cryptographic, parsing, memory-management, or trust-boundary code is touched.
Changed components
src/ui/gui_widgets/multi/web3/gui_connect_wallet_widgets.cInspect captured patch +1 / −6
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 3597011..fa6bb0a 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
@@ -95,7 +95,7 @@ static const lv_img_dsc_t *g_backpackWalletCoinArray[3] = {
};
static const lv_img_dsc_t *g_keystoneWalletCoinArray[] = {
- &coinBtc, &coinEth, &coinTrx, &coinXrp, &coinBnb, &coinLtc, &coinDoge
+ &coinBtc, &coinEth, &coinTrx, &coinXrp, &coinBnb, &coinLtc, &coinDoge, &coinZec
};
static const lv_img_dsc_t *g_UniSatCoinArray[5] = {
@@ -118,7 +118,6 @@ static const lv_img_dsc_t *g_wanderCoinArray[1] = {
static const lv_img_dsc_t *g_beaconCoinArray[2] = {
&coinAr,
- // todo add ao
&coinAo,
};
@@ -425,7 +424,6 @@ static void GuiOpenARAddressNoticeWindow()
lv_obj_align_to(img, lv_obj_get_child(g_noticeWindow, 1), LV_ALIGN_TOP_RIGHT, -36, 36);
}
-
static void GuiOpenUsbNoticeWindow(WALLET_LIST_INDEX_ENUM walletIndex)
{
char connectionDesc[128];
@@ -443,7 +441,6 @@ static void GuiOpenUsbNoticeWindow(WALLET_LIST_INDEX_ENUM walletIndex)
lv_obj_add_event_cb(btn, CloseHintBoxHandler, LV_EVENT_CLICKED, &g_noticeWindow);
}
-
static void OpenQRCodeHandler(lv_event_t *e)
{
WalletListItem_t *wallet = lv_event_get_user_data(e);
@@ -880,7 +877,6 @@ static void GuiCreateQrCodeWidget(lv_obj_t *parent)
// GuiCreateSupportedNetworks(g_connectWalletTileView.walletIndex);
}
-
static void AddFewchaCoins()
{
lv_obj_add_flag(g_bottomCont, LV_OBJ_FLAG_CLICKABLE);
@@ -901,7 +897,6 @@ static void AddFewchaCoins()
}
}
-
static void AddCoinsFromArray(const lv_img_dsc_t *coinArray[], uint32_t arraySize,
bool showMore, int moreOffsetX)
{
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.