Skip to content

Consolidate UI and fee selector#405

Merged
gemcoder21 merged 4 commits into
mainfrom
worktree-nft-unverified-android
May 29, 2026
Merged

Consolidate UI and fee selector#405
gemcoder21 merged 4 commits into
mainfrom
worktree-nft-unverified-android

Conversation

@DRadmir
Copy link
Copy Markdown
Collaborator

@DRadmir DRadmir commented May 29, 2026

  • NFT details now show the verification status (Unverified / Suspicious).
  • NFT links now show the domain (e.g. Website → ens.domains).
  • Network fee screen: consistent amount formatting and a fiat value under each speed option.
  • Asset selection list now uses the short balance format.

Closes #402
Closes #401
Closes #399
Closes #397

Screenshot 2026-05-29 at 1 37 21 PM Screenshot_1780049514 Screenshot_1780049538

DRadmir added 4 commits May 29, 2026 12:39
Show a tappable unverified/suspicious status row on the NFT details
screen, matching iOS. Extract the asset details status row into a shared
verificationStatusItem in :ui keyed on VerificationStatus and reuse it
for both NFT and asset details.

Issue: #402
Display the link host as a value for Website links in the NFT details
Links section, matching iOS. Extract the shared AssetLink -> social-link
mapping (label, icon, website-only host, gemstone ordering) into :ui as
toSocialLinks() and reuse it for both NFT and asset details.

Issue: #401
Align the fee-speed sheet on both platforms:
- iOS: show fiat under gwei/satVb rates (previously native-only).
- Android: native rates show the crypto fee amount + fiat (was fiat-only),
  and the fee total uses the auto value formatter to match iOS.

Issue: #399
Switch the shared cryptoFormatted to ValueFormatter.Style.Short so asset
list balances (select asset, price alerts, staking, buy) match iOS, which
uses .short for these lists.

Issue: #397
@DRadmir DRadmir self-assigned this May 29, 2026
@gemcoder21 gemcoder21 merged commit 6e40e53 into main May 29, 2026
3 checks passed
@gemcoder21 gemcoder21 deleted the worktree-nft-unverified-android branch May 29, 2026 18:07
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.

Add unverified status for nft Missing domain in nft Links Wrong amount formatter for Network Fee Wrong amount formatter on Select Asset screen

2 participants