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

add "max" button to amount fields #2670

Closed
jbibla opened this issue May 31, 2019 · 8 comments · Fixed by #2956
Closed

add "max" button to amount fields #2670

jbibla opened this issue May 31, 2019 · 8 comments · Fixed by #2956
Assignees
Labels
ux-improvement issues that would improve the user experience

Comments

@jbibla
Copy link
Collaborator

jbibla commented May 31, 2019

Is your feature request related to a problem? Please describe.

Often, users want to do make a tx with all the atoms. since there are 6 decimal places, this can be tedious to type out.

Describe the solution you'd like

It is common in these situations to include a max button, which will automagically add the total amount left to the amount field for the user.

Additional context

Something to consider is the relationship with #2674.

@faboweb faboweb added the ux-improvement issues that would improve the user experience label Jun 3, 2019
@stale
Copy link

stale bot commented Aug 23, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 23, 2019
@jbibla
Copy link
Collaborator Author

jbibla commented Aug 26, 2019

simple UI feature. would be nice to have!

@stale stale bot removed the stale label Aug 26, 2019
@mariopino mariopino self-assigned this Sep 3, 2019
@mariopino
Copy link
Contributor

Set Max button is working on branch mario/2670-add-max-amount.

Currently set the entire available balance, so want to discuss about best way to subtract the fee from the available balance. If we click on the button, in the step 2 the fee is 0.000000 ATOM, is this a validation error?

It would be cool to include a fee slider to let user set the fee they want. We can include it in step 1 to let us calculate max = (available balance - fee).

@jbibla
Copy link
Collaborator Author

jbibla commented Sep 5, 2019

Set Max button is working on branch mario/2670-add-max-amount.

awesome! if you make a draft PR we can see the link to the build and test out the feature 😉

If we click on the button, in the step 2 the fee is 0.000000 ATOM, is this a validation error?

no this is actually correct. if the user has no ATOMs left to pay fees we let them submit a tx without fees.

can you add a warning to this section that explains this? i will help with the language.

discuss about best way to subtract the fee from the available balance.

good thought here. a fee slider could work. i don't know what the best approach is.

we could...

  1. allow the user to set the true max with the max button on step 1
  2. have a fee selector on step 1 (either a slider or three button/options)

alternatively...

  1. allow the user to set the true max with the max button on step 1
  2. show a warning that they are not leaving any for fees which may be required
  3. turn the button into a "subtract fees" button

@faboweb what do you think?


this is related to #2530

@faboweb
Copy link
Collaborator

faboweb commented Sep 5, 2019

We decided against a fee selector in favor of less mental overhead for the user.
We could run the simulation already on step 1 to know the fees there to calculate the max. Never the less we need to show a warning that the max already has subtracted fees.
But I vote for show a warning that they are not leaving any for fees which may be required. It is the least complex. And this way we don't block the user in their intended behavior but educate him about the consequences.

@jbibla
Copy link
Collaborator Author

jbibla commented Sep 5, 2019

so what you're proposing is:

  1. when the user clicks max we run the fee simulation and subtract this from the max?
  2. show a message explaining this is what happened and why

are you opposed to the following proposal?

  1. allow the user to set the true max with the max button on step 1
  2. show a warning that they are not leaving any for fees which may be required
  3. turn the button into a "subtract fees" button

@faboweb
Copy link
Collaborator

faboweb commented Sep 5, 2019

Why do you want to make it more complex? Do we want to force the user to pay for fees?

@jbibla
Copy link
Collaborator Author

jbibla commented Sep 5, 2019

my intention is to make it as simple as possible while also letting the user know what's happening.

i am in favour of the simplest solution - which i guess at this point is having a max button that doesn't know anything about fees and a message (#2530) that says "hey, you should probably leave some left over for fees".

that would be simplest and easiest IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux-improvement issues that would improve the user experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants