Skip to content

feat: use dialog for add token button#73

Open
farolanf wants to merge 5 commits into
temptemp3:mainfrom
farolanf:use-dialog-for-add-token-button
Open

feat: use dialog for add token button#73
farolanf wants to merge 5 commits into
temptemp3:mainfrom
farolanf:use-dialog-for-add-token-button

Conversation

@farolanf
Copy link
Copy Markdown
Contributor

Fix #42

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Dec 21, 2023

Screenshot 2023-12-21 at 16 21 10

These move the message down so that there is not a jump in the ui when it appears and disappears.

Maybe what we can do is not have a error message every time an asset is not found.

  • Update asset not found error message to be a less jump ux

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Dec 21, 2023

Screenshot 2023-12-21 at 16 37 13
  • Update add confirmation for vsa
    I am thinking we would need to show some asset info and the confirmation should say "Opt in to {ASSETNAME} (VSA:{ASSETID})

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Dec 21, 2023

Screenshot 2023-12-21 at 16 43 03

Unable to add asset. After clicking add button nothing happens but there is a failed request.

Asset id: 1290865

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Dec 21, 2023

  • Adding arc200 might require some fixing as well
    ARC200:6795477 TacoCoin

@temptemp3
Copy link
Copy Markdown
Owner

Request changes. Thx!

@farolanf
Copy link
Copy Markdown
Contributor Author

Maybe what we can do is not have a error message every time an asset is not found

How about this? (No UI jumps)
Screenshot 2023-12-22 174013

Or should we remove the error message?

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Dec 22, 2023 via email

@farolanf
Copy link
Copy Markdown
Contributor Author

When adding the asset by invoking acc.tokenAccept(tokenId) the WalletConnect QRCode modal pops up which is different with the wallet I use to sign in(WalletConnectModalSign).

With the WalletConnect QRCode I'm unable to sign in
Screenshot 2023-12-25 083508

It works with WalletConnectModalSign
Screenshot 2023-12-25 083612

I think it should use whatever wallet provider the user used to sign in, but I haven't found a way to do it. Any directions?

@temptemp3
Copy link
Copy Markdown
Owner

When adding the asset by invoking acc.tokenAccept(tokenId) the WalletConnect QRCode modal pops up which is different with the wallet I use to sign in(WalletConnectModalSign).

With the WalletConnect QRCode I'm unable to sign in Screenshot 2023-12-25 083508

It works with WalletConnectModalSign Screenshot 2023-12-25 083612

I think it should use whatever wallet provider the user used to sign in, but I haven't found a way to do it. Any directions?

I would look at components/SwapForm it has support for a custom wallet (kibisis) and defly (A-wallet)

@temptemp3
Copy link
Copy Markdown
Owner

I would also pull in changes from main. It has a web3 modal that looks like it is for A-wallet.

@farolanf
Copy link
Copy Markdown
Contributor Author

Demo

demo.mp4

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Jan 10, 2024 via email

@temptemp3
Copy link
Copy Markdown
Owner

Demo

demo.mp4

Could you make a demo for voitest adding an arc200?

@farolanf
Copy link
Copy Markdown
Contributor Author

farolanf commented Jan 16, 2024

Tried adding asset with Kibisis on Voi Testnet but got error:

OperationCanceledError: no response from wallet for "message.enable_request"

Screenshot 2024-01-16 095129

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Jan 16, 2024 via email

@farolanf
Copy link
Copy Markdown
Contributor Author

Kibisis 1.9.0

Also tried after merging with feature/swap, same results.

@temptemp3
Copy link
Copy Markdown
Owner

temptemp3 commented Jan 16, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bounty: UI Improvement: Use dialog instead of window.prompt for Add Token button

2 participants