search

MetaMask / metamask-snaps-beta

Fork of MetaMask that supports plugins! Read the Wiki!
https://github.com/MetaMask/metamask-snaps-beta/wiki
MIT License
142 stars 57 forks source link

Long snap urls breaks permission screen #158

Closed mpetrunic closed 2 years ago

mpetrunic commented 4 years ago

Describe the bug A clear and concise description of what the bug is. If snap is using long urls as origin (ipfs in this case) it breaks UI of confirmation screen

To Reproduce Steps to reproduce the behavior:

  1. Install snap using origin: "https://ipfs.infura.io/ipfs/QmXTu32iPygL2LTvidc4YkpHCoTarDZS2Ks6fa7LVwPVD3"
  2. Connect

Expected behavior A clear description of what you expected to happen. Popup should resize and/or wrap long data.

Screenshots If applicable, add screenshots to help explain your problem. image

Browser details (please complete the following information):

Additional context Add any other context about the problem here.

rekmarks commented 2 years ago

This should hopefully not be the case in the current system, but if it is, an issue should be created on the metamask-extension repo.