HomePhabricator

[Cashtab] Do not wordbreak break-all for XECX summaries

Description

[Cashtab] Do not wordbreak break-all for XECX summaries

Summary:
We use "break-all" for app actions as, most of the time, the content is unknown; it depends on what a user puts in a tx field. So we can have long strings like addresses that we need to "break-all" on, otherwise we get huge overflows that make the app look broken.

I have been seeing screenshots of "break-all" applied to XECX reward summaries which looks dumb. This is a hard-coded string and we do not need to worry about overflow issues.

So, do not "break all" for this specific app action. Add a prop that can control this behavior in other app actions as we uncover others that have this same issue.

Test Plan:
npm test, screenshots

before:

image.png (146×490 px, 21 KB)

after:

image.png (149×523 px, 21 KB)

Reviewers: #bitcoin_abc, emack

Reviewed By: #bitcoin_abc, emack

Differential Revision: https://reviews.bitcoinabc.org/D17628

Details

Provenance
bytesofmanAuthored on Sun, Feb 2, 19:39
bytesofmanPushed on Mon, Feb 3, 03:37
Reviewer
Restricted Project
Differential Revision
D17628: [Cashtab] Do not wordbreak break-all for XECX summaries
Parents
rABC9b9a406c97e4: [Cashtab] Make sure hide balance toggle also hides XECX balance
Branches
Unknown
Tags
Unknown