Page MenuHomePhabricator

[Cashtab] add eToken icon to send page
AbandonedPublic

Authored by kieran709 on Dec 2 2021, 18:12.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Summary

Using the same logic as TokenListItem, eToken icon added to send page. Related to task T1988.

Custom Token:

resize-token-icons-send-page.png (1×1 px, 356 KB)

Generated Token:

resize-token-icons-send-page-2.png (1×1 px, 250 KB)

Test Plan

cd web/cashtab
npm start
navigate to eTokens tab
select a token with a custom icon
ensure icon displays at top of token table
navigate back to eTokens tab
select a token with a generated icon
ensure icon displays at top of token table

Diff Detail

Repository
rABC Bitcoin ABC
Branch
display-icon-etoken-page
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17544
Build 34914: Build Diffcashtab-tests
Build 34913: arc lint + arc unit

Event Timeline

bytesofman requested changes to this revision.Dec 2 2021, 23:05
bytesofman added inline comments.
web/cashtab/src/components/Send/SendToken.js
365 ↗(On Diff #31236)

Just "Icon" for the label

368 ↗(On Diff #31236)

Other available sizes here are 64, 128, 256, and 512

Try all of them and use the biggest one that works

This revision now requires changes to proceed.Dec 2 2021, 23:05

increased size to 256, the biggest size that will fit.

bytesofman requested changes to this revision.Dec 3 2021, 22:33
bytesofman added inline comments.
web/cashtab/src/components/Send/SendToken.js
374 ↗(On Diff #31258)

If the token does not have a custom icon, keep it 32x32

height="32"

375 ↗(On Diff #31258)

width="32"

This revision now requires changes to proceed.Dec 3 2021, 22:33

responding to review feedback, generated icons are now 32px.

This revision is now accepted and ready to land.Dec 8 2021, 15:49