Repository navigation
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Closing as a duplicate of #1565 (@ben-kaufman), which already gives the USDT wallet its opaque background and says Fixes #1519. The one extra here, |
[Low impact] The PR appears safe to merge. SummaryThe USDT wallet now paints a black background and blocks taps from reaching Home.
Reviews (1) · Last reviewed commit: "fix: slide the usdt wallet in like the o..." · Reviewed by Greptile |
Regtest APKDownload bitkit-dev-debug universal APK (expires in 30 days). |
Closes #1519
Description
Out of Scope
UsdtWalletScreensheets: the Receive › USDT › Edit transition is bug: receive usdt edit sheet screen has no transitions #1586.Design
N/A — no design change. The screen keeps its
Savings, Spending, Hardware, USDT Balances › Wallet USDT (funds & activity)frame; only its background and transition are fixed.Preview
Before: USDT fades in over Home
before-2x.mp4
After: USDT slides in and out like Spending
after-2x.mp4
Both recorded on an Android 16 emulator with animations at 3x duration, played at 2x. Each opens USDT, goes back, then opens Spending for comparison.
QA Notes
Journeys
usdt-wallet-navigation.xml— USDT opens and closes with the same slide as Savings and Spending, with no Home content showing throughManual Tests
N/A
Automated Checks
N/A