Page MenuHomePhabricator

[Cashtab] Prevent content jumping on receive screen on input switch with tighter validation msgs
ClosedPublic

Authored by bytesofman on Oct 20 2025, 23:54.

Details

Summary

If the user switches from XEC to FIRMA, they see a validation error bc 0 is not allowed. This is annoying but maybe necessary with current workflow, as bip21 requires SOME value there. So we would need to pick a default to avoid the error.

The real annoying thing though is the content jumping.

Trying to solve this, saw a lot of these validation errors are overly long and complex. Not really useful for the user so much as dev notes.

So, tighten up the validation msgs to fix the content jumping and improve concise UX. And add some space for the validation error to appear without causing content jumping.

Test Plan

npm test, npm start and try it out

Diff Detail

Repository
rABC Bitcoin ABC
Branch
no-insta-invalid
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 34834
Build 69130: Build Diffcashtab-tests
Build 69129: arc lint + arc unit

Event Timeline

bytesofman retitled this revision from [Cashtab] Prevent auto validation error on receive screen on input switch to [Cashtab] Prevent content jumping on receive screen on input switch with tighter validation msgs.
bytesofman edited the summary of this revision. (Show Details)
Fabien added a subscriber: Fabien.

I didn't check for content jump but the error messages are much better

This revision is now accepted and ready to land.Wed, Oct 22, 07:34