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

Coin control interface #261

Merged
merged 9 commits into from
Dec 3, 2020

Conversation

song50119
Copy link
Contributor

This PR about coin control features
And this is depends on this PR

Screen Shot 2020-11-03 at 19 58 58

Screen Shot 2020-11-03 at 19 59 06

@song50119 song50119 force-pushed the coin-control-interface branch 2 times, most recently from 7cf34e9 to bd266fb Compare November 3, 2020 13:23
@song50119 song50119 marked this pull request as ready for review November 20, 2020 13:55
@oshorefueled
Copy link
Contributor

"balance after send" seems to be showing me the remainder of my selected inputs after send. It's confusing as it is in conflict with my main balance.
Screenshot 2020-11-27 at 08 06 00

Amount selected is higher than available balance. 232.99 DCR is displayed as my wallet balance. However my selected amount is 247.999.
Screenshot 2020-11-27 at 10 21 56

I have no inputs selected but the title row shows a quantity of 2 and 28 DCR. Also I feel "selected" would be a better title than "quantity".
Screenshot 2020-11-27 at 10 22 33

* Handler inputs checkboxes
* Display amount and fee to layout
* Adds (*pageCommon).selectedUTXO
* Adds select all button
* Adds selecAllChexBox
* Adds SendPage.unspentOutputsSelected
* Adds SendPage.customChangeAddressEditor
* Broadcast Txn when coin control is enable
* Refactor (*SendPage).coinControlFeatured
@song50119 song50119 force-pushed the coin-control-interface branch from a267b2a to 045e929 Compare December 2, 2020 13:26
ui/send_page.go Outdated Show resolved Hide resolved
- Clear selected utxo after broadcast txn
@song50119 song50119 force-pushed the coin-control-interface branch from eea93fb to d0bde41 Compare December 3, 2020 14:27
* change (*SendPage).coinControlFeatured to (*SendPage).coinControlLayout
@oshorefueled oshorefueled merged commit 15f1eac into planetdecred:master Dec 3, 2020
@song50119 song50119 deleted the coin-control-interface branch December 18, 2021 09:40
@song50119 song50119 restored the coin-control-interface branch December 18, 2021 09:40
song50119 pushed a commit to song50119/godcr that referenced this pull request Apr 24, 2022
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.

2 participants