Display & send individual collectibles

[bdresser]

@estebanmino is working on the extension's ability to pull custom images for users' collectibles / ERC 721 tokens. These need a home in the extension.

MVP idea we could keep them on the side-menu with other tokens, then display individual tokens in a scrollable list - either as vertical scroll in a tab next to History, or as a horizontal scroll where the centered icon is?

Longer-term we can consider whether the side-menu should be a feature of the home screen itself (as discussed in Slack) etc.

Collectibles will each have a "name" and an image to start.

[bdresser]

@estebanmino will we also support sending NFTs like any other token?

[estebanmino]

yeah that's the idea

[bdresser]

@estebanmino what data is available per token? We have an image - do we have a name or any other data?

Any other things we should be aware of as @bahnju starts thinking about design?

[estebanmino]

@bdresser @bahnju we have:

• Collectible contract name
• Collectible contract address
• Collectible specific name
• Collectible specific token ID
• Collectible specific image

[bdresser]

Might be worth looking at https://alphawallet.com as well!

[bdresser]

The send flow also needs to be designed for mobile. Separate but related issue MetaMask/website#282

[aakilfernandes]

Hey, just wanted to pipe in here to give my thoughts since we're working on NFTs for GuildCrypt.

1. In general, no-one knows what the symbol of NFTs are. Using the Name instead of the Symbol would be better.

2. "1 CK" is confusing to users. Should be "CryptoKitty Test against two dapp tabs open #1"

3. Not all NFTs implement an image. There's a Metadata standard for the URI of off-chain data, but AFAIK no standard for the form of that data (i.e. JSON, or the keys in the JSON). There's a defacto standard which is implemented by OpenSea

[bdresser]

partially blocked by #5806

[bdresser]

@estebanmino which GABA controller would make this possible in the extension?

[estebanmino]

@bdresser NetworkController and PreferencesController. When we have that we could integrate assets related controllers.

[Garito]

Any advance on this issue?

[bschorchit]

Closing this as it's fixed with NFT support on extension