brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.06k stars 2.23k forks source link

Show Chain icon in confirmation bubble #17639

Open spylogsster opened 2 years ago

spylogsster commented 2 years ago

Description

Api call wallet_addEthereumChain can pass network icon urls, we need to show it in the confirmation bubble

Steps to Reproduce

example of call:
{
  "id": 1,
  "jsonrpc": "2.0",
  "method": "wallet_addEthereumChain",
  "params": [
    {
      "chainId": "0x64",
      "chainName": "xDAI Chain",
      "rpcUrls": ["https://dai.poa.network"],
      "iconUrls": [
        "https://xdaichain.com/fake/example/url/xdai.svg",
        "https://xdaichain.com/fake/example/url/xdai.png"
      ],
      "nativeCurrency": {
        "name": "xDAI",
        "symbol": "xDAI",
        "decimals": 18
      }
    }
  ]
}

Actual result:

Icon is not shown

Expected result:

Icon is shown in bubble

Reproduces how often:

Desktop Brave version:

Android Device details:

Version/Channel Information:

Other Additional Information:

Miscellaneous Information:

bbondy commented 2 years ago

Moving out of MVP

spylogsster commented 2 years ago

in front end side icons can be shown this way https://github.com/brave/brave-browser/issues/20650