Skip to content

ADD: Highlight first and last segments of bitcoin address#8372

Merged
marcosrdz merged 1 commit intomasterfrom
enhance-address
Mar 4, 2026
Merged

ADD: Highlight first and last segments of bitcoin address#8372
marcosrdz merged 1 commit intomasterfrom
enhance-address

Conversation

@ojokne
Copy link
Copy Markdown
Member

@ojokne ojokne commented Mar 4, 2026

Follows #8362

  • Updated color to alternativeTextColor2 to match the color used the bitcoin amount displayed on confirm screen
  • Added address highlighting to the multisig flow
Screenshot_1772620984 Screenshot_1772620996

@ojokne ojokne added the testflight Upload iOS and macOS builds to Testflight. label Mar 4, 2026
Copy link
Copy Markdown
Collaborator

@GladosBlueWallet GladosBlueWallet left a comment

Choose a reason for hiding this comment

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

Test results: I found edge cases. Try not to trip on them.

destinationAddressView.push(
<View style={styles.destinationTextContainer} key={`end-${index}`}>
<Text numberOfLines={0} style={[styles.textDestinationFirstFour, stylesHook.textFiat]}>
<Text numberOfLines={0} style={[styles.textDestinationFirstSix, stylesHook.textFiat]}>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Minor: numberOfLines={0} is a weird no-op (and platform-dependent). If you want unlimited lines, omit numberOfLines; if you want truncation, set a real limit + ellipsizeMode.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

✅ Build 7.2.7 (1772618571) has been uploaded to TestFlight and will be available for testing soon.

@ojokne ojokne force-pushed the enhance-address branch from cf4f26a to a3a850c Compare March 4, 2026 10:50
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

✅ Build 7.2.7 (1772621917) has been uploaded to TestFlight and will be available for testing soon.

@ojokne ojokne marked this pull request as ready for review March 4, 2026 15:21
@marcosrdz marcosrdz merged commit 965f1a9 into master Mar 4, 2026
14 checks passed
@marcosrdz marcosrdz deleted the enhance-address branch March 4, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testflight Upload iOS and macOS builds to Testflight.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants