Page MenuHomePhabricator

[Cashtab][Alias] Increase pending warning emphasis on confirmation modal
ClosedPublic

Authored by emack on Nov 2 2023, 02:01.

Details

Summary
  • The title is now conditionally rendered: it's Confirm alias registration when the alias is available with no pending txs, otherwise it's a Warning title with icon.
  • Modal body text also conditionally rendered based on existance of pending txs for this alias, with color scheme updated to use green for alias availability and the default alert color for the pending warning
Test Plan
  • Register a new alias and verify the confirmation dialog only has the confirmation message
  • Register the same new alias and confirm the pending substrings and UI make match the following:

When alias is available with no pending registrations onchain

image.png (266×573 px, 27 KB)

When alias is available with existing pending registrations from other users but not from this wallet

image.png (294×547 px, 28 KB)

When alias is available with existing pending registrations which includes registrations from this wallet

image.png (308×544 px, 32 KB)

Alias available, there are pending txs including from this wallet, and registration is on behalf of another address

image.png (375×554 px, 43 KB)

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emack requested review of this revision.Nov 2 2023, 02:01
emack edited the test plan for this revision. (Show Details)
Fabien requested changes to this revision.Nov 2 2023, 11:20
Fabien added a subscriber: Fabien.

In the case there are pending alias you should not have the green "everything is fine" register confirmation text, but rather something purple like "Continue the registration anyway ?"

This revision now requires changes to proceed.Nov 2 2023, 11:20
emack edited the summary of this revision. (Show Details)
emack edited the test plan for this revision. (Show Details)

Updated UI as follows

Pending alias including 1 for this address

image.png (289×542 px, 27 KB)

Pending alias including 1 that’s not from this address

image.png (257×539 px, 22 KB)

Pending alias from this wallet + registering on behalf of another address

image.png (329×536 px, 35 KB)

No change to no pending UI

This revision is now accepted and ready to land.Nov 3 2023, 13:46