snaps icon indicating copy to clipboard operation
snaps copied to clipboard

Display the Snap's Icon in Snap Simulator.

Open mario-christopher opened this issue 1 year ago • 0 comments

When a Builder creates a Snap and submits it to us to allowList, we usually have no way to preview the Snap's Icon. Its only after allowListing, when the Snap shows up in the Directory, that we notice some discrepancies in the Icon. Sometimes, the builder forgets about creating their icon and instead has the default icon in there.

Can we add an Icon Preview in the Snaps Simulator ? In this way, the onus is on the Builder to have a proper Icon created and they can preview and confirm it, before they submit the snap to us to allowList.

Additional Feedback from Builder: It would be helpful if the snap simulator and/or build command checked the validity of the SVG (i.e., this check: https://github.com/MetaMask/snaps/blob/main/packages/snaps-sdk/src/internals/svg.ts#L41). The SVG displayed fine for me in Chrome but the snap validation is stricter. In any case, we didn't expect an invalid SVG to cause the entire snap to be un-installable.

mario-christopher avatar Mar 04 '24 15:03 mario-christopher