client icon indicating copy to clipboard operation
client copied to clipboard

Metamask is 'unknown' when Frame is present

Open paalex opened this issue 5 years ago • 3 comments

Describe the bug Metamask connect is not available when Frame Chrome extension is installed

Mainnet or testnet? testnet

Organization https://rinkeby.aragon.org/#/tvpdaotest

To Reproduce Steps to reproduce the behavior:

  1. Go to https://rinkeby.aragon.org/#/<dao-name-here>
  2. Click on 'Connect account' and connect Metamask
  3. Install or enable Frame extension
  4. Reload the page
  5. See that Metamask is 'unknown' in 'Connect account'

Expected behavior Metamask is available for connect

Screenshots Screen Shot 2020-06-16 at 10 45 09 PM

Desktop:

  • macOS Mojave 10.14.6
  • Chrome 83.0.4103.106 (Official Build) (64-bit)

Additional context The Agent app requires Frame, which conflicts with Metamask, making it impossible to keep using Metamask. Disabling Frame extension and reloading the page makes Metamask work again. Enabling and reloading breaks Metamask. Might be related to #1320, #1364, #161

paalex avatar Jun 17 '20 02:06 paalex

This is also related to https://github.com/aragon/aragon/issues/1389, where we don't detect a non-Metamask injected provider very well. In this case, if you have the Frame extension enabled, that first panel should show something like Frame (extension).

The Agent app requires Frame, which conflicts with Metamask, making it impossible to keep using Metamask.

However, it should be noted that if you want to use the Frame extension, you should also disable Metamask. Only one injected provider should be active at a time.

sohkai avatar Jun 17 '20 07:06 sohkai

Hi @sohkai, if it is implemented like you suggested, it would look like the following, i.e. Frame and Frame (Extension) would show up and clicking on any of them would connect Frame. Wouldn't it be preferable to hide one of these buttons? image

FelixMohr avatar Jun 20 '20 15:06 FelixMohr

Yes, true—we could hide the button for the extension in that case!

sohkai avatar Jun 23 '20 19:06 sohkai