-
Notifications
You must be signed in to change notification settings - Fork 10
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
withdrawal/send: Support selecting custom sat per byte #447
Comments
Should it be a simple input, a slider? |
Instead of slider I would like to use large buttons.
Please let me know if you would like a different direction @Janaka-Steph |
Agreed. We can re-use the same modal component used in Settings > L-BTC Unit (without the search box) |
Perfect. I've been getting familiarized with the codebase and setting up the environment these past two days. I'll submit a PR tomorrow morning to track the progress. In the meantime I'll send you guys some screenshots of the UI I'm working on. I like this approach because:
|
Maybe better to use the same button style as the Confirm button, but with border radius similar to address input field ? |
Thanks for the feedback. I'll do the custom fee like the second screenshot in BW. At the bottom where it says "Personalizada" it means "Custom amount". You click it and then an input dialog pops up. |
I'm done with the second screen. Any feedback on this new screen would be great :D @Janaka-Steph I'll make sure to include the changes you suggested in the first screen. The "NETWORK FREE" label and the "SAT/VBYTE" button I added are actually aligned with the "MAX"/"FIAT VALUE" labels above. I'll change the alignment and test out the other changes as well. |
At the moment we hardcoded to 0.1 sat per byte, if Liquid Network usage grows we must let the user decide
The text was updated successfully, but these errors were encountered: