fix(frontend): keep Treasury Assets values readable on iPad #38
No reviewers
Labels
No labels
bug
docs
documentation
duplicate
enhancement
frontend
good first issue
help wanted
invalid
missing-implementation
priority
medium
product
QA
question
testing
UX
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
code/ustr-cmm!38
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "21-fix-treasury-assets-ipad-layout"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes #21.
Summary
lg), and 3-col only atxl(1280+). The loading skeleton uses the sameTREASURY_ASSETS_GRID_CLASS.xl) and wrap primary / USD / CR /NAV incompleteinstead of CSS ellipsis. iPad has no hover, sotitleis not the disclosure.CR $0.00 · UST1, cUSTC omitted).formatPoolSharerounding is unchanged. Card-only large-spot display decimals do not feed CR/NAV.treasuryAssetDisplay.tswith unit tests. Invariants and playbook:skills/frontend-treasury-assets-layout/SKILL.md.Display-only. Diff does not touch
treasuryRatios,lpNav, available-supply, tokenlist allowlist, or on-chain queries.Test plan
cd frontend && npm test && npm run check/treasuryat 375: 1-col, no page X-scroll, full USTC/LUNC amounts/treasuryat 768 / 820 / 834 / 1024: 2-col (not 3 at 1024), full100% of pool/97.78% of pool, full CR haircut lines including$0+ omitted legs/treasuryat 1280: 3-col still readable, no ellipsis/address/{treasury}/address/{pair}· session oracledoes not collide; empty / loading still workust1cmm.com/treasuryon a real iPad (portrait + landscape)Acceptance (#21)
LP UST1/SpaceUSDxlonlyNAV incomplete, vFDUSD session-oracle keptformatPoolSharerounding unchanged (≥ 0.99995→ 100%;0.99994stays 99.99%)#20LP CL8Y/cUSTClabel (wrap, not hardcode)Not done (out of scope or post-deploy)
#20CL8Y/cUSTC pin itself (layout only tolerates the label).mentioned in issue #21
mentioned in merge request !40
Superseded by !40 (
21-treasury-assets-ipad-layout).!38 dropped Playwright
data-testidhooks that master e2e relies on and removed some #18 skill cross-links. !40 keeps those, adds viewport e2e, and is verified at 375/768/834/1024/1280.Pull request closed