[Cashtab] Fix formatting issue for Cashtab msgs containing long strings
ClosedPublic
Actions

Authored by kieran709 on Oct 24 2022, 16:24.

Details

Reviewers

bytesofman
emack

Group Reviewers

Restricted Project

Commits

rABCc2a65a075c0d: [Cashtab] Fix formatting issue for Cashtab msgs containing long strings

Summary

Related to T2755. Long unbroken strings were pushing out the OpReturnType div beyond the boundaries of the app.

Test Plan

cd web/cashtab && npm start
ensure there is a tx with a long unbroken string in recent tx history
observe that the message wraps instead of breaking out of the containing div

Diff Detail

Repository

rABC Bitcoin ABC

Branch

fix-formatting-for-long-messages

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20687
Build 41038: Build Diff	cashtab-tests
Build 41037: arc lint + arc unit

Event Timeline

kieran709 created this revision.Oct 24 2022, 16:24

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 24 2022, 16:24

kieran709 requested review of this revision.Oct 24 2022, 16:24

Harbormaster completed remote builds in B20687: Diff 35968.Oct 24 2022, 16:27

bytesofman accepted this revision.Oct 24 2022, 18:00

This revision is now accepted and ready to land.Oct 24 2022, 18:00

emack accepted this revision.Oct 25 2022, 07:21

Closed by commit rABCc2a65a075c0d: [Cashtab] Fix formatting issue for Cashtab msgs containing long strings (authored by kieran709). · Explain WhyOct 25 2022, 16:49

This revision was automatically updated to reflect the committed changes.

kieran709 added a commit: rABCc2a65a075c0d: [Cashtab] Fix formatting issue for Cashtab msgs containing long strings.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

Home/

Tx.js

3 lines

Diff 35968

View Options

web/cashtab/src/components/Home/Tx.js

[Cashtab] Fix formatting issue for Cashtab msgs containing long stringsClosedPublicActions