HomePhabricator

[Cashtab] Improve UX for sending cashtab msg

Description

[Cashtab] Improve UX for sending cashtab msg

Summary:
T2207

Stop using the confusing below-send collapse for a cashtab msg. Instead, use a simple switch to show/hide the form.

Put cashtab msg into formdata state object instead of maintaining its own confusingly named state var (confusing bc now cashtabmsg is not the only type of opreturn msg we could expect to send)

Test Plan:
npm test

This diff is live at https://cashtab-local-dev.netlify.app/

note: there is a pre-existing bug where the send button will not be disabled on multisend if there is valid input present in send to one

image.png (437×564 px, 31 KB)

image.png (633×550 px, 61 KB)

image.png (705×550 px, 58 KB)

image.png (705×550 px, 59 KB)

error and count if error msg is more than 1 line

image.png (330×290 px, 26 KB)

Reviewers: #bitcoin_abc, emack

Reviewed By: #bitcoin_abc, emack

Subscribers: emack

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

Details

Provenance
bytesofmanAuthored on Apr 1 2024, 19:54
bytesofmanPushed on Apr 2 2024, 23:20
Reviewer
Restricted Project
Differential Revision
D15844: [Cashtab] Improve UX for sending cashtab msg
Parents
rABCddfa35072960: [website] Add links to FAQ
Branches
Unknown
Tags
Unknown