fix(contacts): name the Contacts sweep dir + light-skin badge contrast
Screenshot-sweep review follow-ups: - sweepPageName() had its own page-name switch that lacked a Contacts case, so the tab's screenshots landed under the fallback "page/" dir. Add the case so they capture under "contacts/". - The Z/T type badge (green/amber) washed out on light skins. Use darker, more-saturated variants when IsLightTheme(); keep the brighter ones on dark. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -757,6 +757,7 @@ static const char* sweepPageName(ui::NavPage page)
|
||||
case ui::NavPage::Send: return "send";
|
||||
case ui::NavPage::Receive: return "receive";
|
||||
case ui::NavPage::History: return "history";
|
||||
case ui::NavPage::Contacts: return "contacts";
|
||||
case ui::NavPage::Mining: return "mining";
|
||||
case ui::NavPage::Market: return "market";
|
||||
case ui::NavPage::Console: return "console";
|
||||
|
||||
@@ -312,10 +312,12 @@ void RenderContactsTab(App* app)
|
||||
|
||||
ImGui::TableNextColumn();
|
||||
// Z/T type badge + address in normal (legible) text — not muted.
|
||||
// Darker/more-saturated variants on light skins so the badge doesn't wash out.
|
||||
bool shielded = isShieldedAddr(entry.address);
|
||||
ImGui::TextColored(shielded ? ImVec4(0.35f, 0.80f, 0.60f, 1.0f)
|
||||
: ImVec4(0.95f, 0.72f, 0.30f, 1.0f),
|
||||
"%s", shielded ? "Z" : "T");
|
||||
bool lightTheme = material::IsLightTheme();
|
||||
ImVec4 zCol = lightTheme ? ImVec4(0.10f, 0.55f, 0.38f, 1.0f) : ImVec4(0.35f, 0.80f, 0.60f, 1.0f);
|
||||
ImVec4 tCol = lightTheme ? ImVec4(0.72f, 0.48f, 0.05f, 1.0f) : ImVec4(0.95f, 0.72f, 0.30f, 1.0f);
|
||||
ImGui::TextColored(shielded ? zCol : tCol, "%s", shielded ? "Z" : "T");
|
||||
ImGui::SameLine(0.0f, 6.0f);
|
||||
std::string addr_display = entry.address;
|
||||
int addrTruncLen = (addrTable.columns.count("address") && addrTable.columns.at("address").truncate > 0) ? addrTable.columns.at("address").truncate : 40;
|
||||
|
||||
Reference in New Issue
Block a user