Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

rework UI for coin control #3130

Merged
merged 9 commits into from
Sep 4, 2020
Merged

rework UI for coin control #3130

merged 9 commits into from
Sep 4, 2020

Conversation

valpinkman
Copy link
Member

@valpinkman valpinkman commented Aug 21, 2020

Type

UI Improvement

Context

LL-1684

Parts of the app affected / Test plan

Coin Control
image
Screenshot 2020-08-21 at 16 19 34
image

image

@github-actions
Copy link

github-actions bot commented Aug 21, 2020

PR to merge feat/coin-control 54ff8d6 -> develop

Diff output

Please download attached snapshots.zip, extract it in folder and update your PR with it

Copy link
Contributor

@dasilvarosa dasilvarosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work! Some minor remarks:

Send step 2 (Amount)

  • Change order of the toggles: RBF first.
  • RBF enabled by default.
  • Reword (add a comma): “Include coins from unconfirmed, replaceable transactions”
  • Reword: "Allow speeding up the transaction later by increasing the network fees (Replace-By-Fee)"
  • Add infobubble on "Coin control" CTA hover: "You must first enter an amount." Figma

Select coins modal

  • Reword title: Select coins --> Coin control
  • DOGE 10 —> 10 DOGE to be consistent with the new way of displaying units
  • More spacing between CTA and support link
  • Reword strategies: “Oldest coins first (FIFO)“, “Minimize fees (optimize size)“, “Minimize future fees (merge coins)”
  • Slight reword: "You cannot select coins from unconfirmed, replaceable transactions."
  • Implement not enough coins selected error case like in the design. Make sure it's stickied when scrolling down the list of inputs. Also make sure the "Coins to spend" / "Change to return" do not disappear and display a "-" like in the design. Figma
  • Reword: "INPUT 0" --> "SELECTED" (more intuitive UX)

@valpinkman
Copy link
Member Author

valpinkman commented Aug 25, 2020

Implement not enough coins selected error case like in the design. Make sure it's stickied when scrolling down the list of inputs.
@dasilvarosa seen with Chris, we keep the same banner for now, as there will be a new task to update them.

  • sticky done

I believe if we want a new error text, we need to throw a new error type (this involves libcore I think)

dasilvarosa
dasilvarosa previously approved these changes Aug 26, 2020
@Arnaud97234 Arnaud97234 self-assigned this Aug 31, 2020
Copy link
Contributor

@Arnaud97234 Arnaud97234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bugs found

  1. In the coin control modal -> Pending tx: "Pending" and "Replaceable" text should be in Red.

Actual
actual

Expected
expected



  1. UTXOs shouldn't be displayed twice

Screenshot 2020-08-31 at 19 39 47

___________ ___________
  1. Maximum spendable should adapt when user turns OFF "Include coin from unconfirmed / replaceable tx" AND there are some UTXOs corresponding to RBF tx.
  • if OFF -> Max spendable = Balance - Replaceable UTXOs

ezgif com-video-to-gif

Copy link
Contributor

@Arnaud97234 Arnaud97234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the coin control modal -> Pending tx: "Pending" and "Replaceable" text should be in Red.

Actual
actual

Expected
expected

Arnaud97234
Arnaud97234 previously approved these changes Sep 1, 2020
@Arnaud97234 Arnaud97234 removed their assignment Sep 1, 2020
dasilvarosa
dasilvarosa previously approved these changes Sep 3, 2020
IAmMorrow
IAmMorrow previously approved these changes Sep 3, 2020
@Arnaud97234 Arnaud97234 self-requested a review September 3, 2020 16:16
Arnaud97234
Arnaud97234 previously approved these changes Sep 3, 2020
@Arnaud97234 Arnaud97234 removed the request for review from nabil-brn September 4, 2020 09:39
@Arnaud97234 Arnaud97234 merged commit a3f39a1 into LedgerHQ:develop Sep 4, 2020
@valpinkman valpinkman deleted the feat/coin-control branch December 15, 2020 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants