Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zephira quality assesment feedback #27

Open
ZephiraDev opened this issue Mar 3, 2023 · 0 comments
Open

Zephira quality assesment feedback #27

ZephiraDev opened this issue Mar 3, 2023 · 0 comments

Comments

@ZephiraDev
Copy link

ZephiraDev commented Mar 3, 2023

Hello. The in browser wallet functioned well and was generally easy to use. For reference you can always view my demo on Youtube. Link down below. now my suggestions for improvement and other commentary
Wallet creation:

When creating a new wallet the seed phrase is stored in local browser data. Maybe some information should be added that clearing the browser cache / web content / cookies will result in loss of the wallet. Also, maybe an option to export the private key to text as a regular user will probably not know how to extract the seed phrase out of local browser data. in case of firefox this data is stored in an SQLite database at the location ~/.mozilla/theActiveProfile/. Additionally maybe a pop up or text to remind the user to encrypt the browser wallet by adding a passphrase.

Staking

When proceeding to stake after the wallet has been funded, the wallet unlock will trigger an accept of terms of service. I first wrongly concluded that Accepting the TOS would already start staking my entire balance. This is obviously not the case as after Accepting the TOS you need to press the stake button again. Maybe some message in between or after the TOS acceptance has been completed that "You are now ready to stake" and maybe some information that a small transaction has been made to the Crowdnode API to accept the TOS, as this is then reflected in the wallet balance. Then when pressing the stake button also in the next screen let the user know that an empty wallet balance will make the unstaking later not possible as you always need balance in your wallet to fund transactions. the unstaking is basically a transaction again.

withdrawing

when a balance is displayed in the wallet that I have 0.02 dash and I make a withdrawal request then input the amount 0.02 to withdraw the application will halt and not proceed. if I let the amount to withdraw empty (to withdraw the entire balance) then "0.02" dash will be withdrawn (obviously - trancastion fees) but you should improve this part where the user can see "0.02" in wallet, I want to withdraw "0.02" so when I enter "0.02" it will correctly take out the entire balance , this is again about fees and such. I would try to improve this partt.

graphics style suggestions

Maybe a dash logo somewhere worked into the background or at the top in the menu somewhere would be a nice addition. maybe a gradient at the top and bottom you can create a 1px*200px gradient for example. the menu bar could be having a different gradient or more distinct color as now everything is just the same color. there are small things you can do to make it look a little better. I really liked the steak image for the staking button :P

Good and easy wallet, together with the CLI tool these have been nice to test!

https://www.youtube.com/watch?v=uuuXDOVxges

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

No branches or pull requests

1 participant