search

TokelPlatform / tokel_dapp

Tokel's All-In-One application. Featuring a coin wallet, token wallet, tokenDEX, NFT marketplace and a token creation page.
https://tokel.io
MIT License
39 stars 24 forks source link

Make seed input a set of discrete input boxes #113

Open gcharang opened 3 years ago

gcharang commented 3 years ago

In both address gen flow and restore flow, when asking a user to input seed words, make the user enter each word in a different box. This way, missing/extra space user errors can be eliminated. the inputs can be trimmed, validated against word list, etc., and can be used inside the app to derive the actual seed phrase

CHMEX commented 3 years ago

Additionally the way it works now to enter the seed is very convenient and works for me as ma matter of fact I love the simplicity. However people will lose their funds with this. As they will enter 2022 and think it's the password for the wallet. when it is actually the seed.

gcharang commented 3 years ago

@CHMEX you mean that the current UI is good for you. but, you think that newbies might get confused and will lose money right?

CHMEX commented 3 years ago

Yes correct for myself its perfect just to enter the wif and it works. However new people will think its a password to protect the wallet so they use 2020 with is the seed. Same we had in agama where 300k was deposited a famous address seed was pretty weak.....

On Sat, Jul 31, 2021, 11:25 gcharang @.***> wrote:

@CHMEX https://github.com/CHMEX you mean that the current UI is good for you. but, you think that newbies might get confused and will lose money right?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/TokelPlatform/tokel_app/issues/113#issuecomment-890318544, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH7AI5SFEUO3KUNBR5WICNLT2O6QBANCNFSM5BESGEEQ .

piggydoughnut commented 3 years ago

ah good point, didnt think about this kind of use case at all. Thx for mentioning it.