search

ethereum / remix-plugins-directory

Remix plugins directory
51 stars 67 forks source link

Celo plugin: using input fields when there is no input required from the user #110

Closed bunsenstraat closed 3 years ago

bunsenstraat commented 3 years ago

Fields such as language, filename, account and the empty input next to the link button are input fields that don't accept inputs. This might be confusing. There are only real input fields in the contract and the 2 dropdowns.

daoauth commented 3 years ago

Yes, I'll fix it. But, account is required. Used when connecting with MetaMask.

bunsenstraat commented 3 years ago

But you can't actually type something in that field? it is automatically filled, so then why should it be an input field? Am I seeing this correctly?

daoauth commented 3 years ago

When a user first uses the plugin, pressing the Connect button will connect to the MetaMask and the account is automatically entered. And if you change the Account in MetaMask, it will also change. Screen Shot 2020-11-05 at 11 34 19 AM

RemixIDE's DEPLOY & RUN TRANSACTIONS plugin's account field also changes according to MetaMask's account change. This is because it was developed with reference to the function of DEPLOY & RUN TRANSACTIONS plugin. And I plan to refer to further development too. Screen Shot 2020-11-05 at 11 34 32 AM

When the user first uses the plugin, I think there should be a button to connect with MetaMask explicitly. So, there seems to be a way to follow the DEPLOY & RUN TRANSACTIONS plugin of RemixIDE or to make the connect button visible only when it is not connected. (However, because of the extension wallet for CELO cannot be used on DEPLOY & RUN TRANSACTIONS plugin of RemixIDE, I think it will be needed an account field and a connect button for the Celo extension wallet.)