Skip to content

Home: explain what token-list changes mean versus the wallet change #486

Closed

Description

@Ryun1

Problem

The home screen shows two kinds of change figure, and users can't tell how they relate:

  • Hero: x% · 24h (home-view.ts). This comes from portfolioDelta in shared/fiat.ts: the 24h price change of current holdings, weighted by value. It rebuilds each holding's value from 24h ago using today's balance, so sends, receives and swaps don't count.
  • Token rows: ↑ x% (changeBadge in token-row.ts). This is each token's own 24h price change, with no window label.

So the token percentages don't add up to the hero figure (it's a value-weighted average). And neither figure reflects changes in the user's balance, which is probably what "increase/decrease" sounds like it means.

Proposed solution

  • Label both as price moves with their window, e.g. hero "Portfolio price · 24h" and rows "24h price".
  • Add a short info line or tooltip: "Price change over 24h at your current balances; sends and receives aren't included."
  • Out of scope unless asked for: a real change in balance-based value (needs balance history).

Additional context

Checked against #450 (feature/mobile-wallet). The logic is the same on main.

Activity

  1. mossbot55 commented on Oct 3, 2026

    @mossbot55
    Contributor

    Picking this up. I will add a short explanation of what the home change figures mean (24h price move at current balances, sends and receives excluded) and confirm the wording against the copy style before opening a PR.

  2. mossbot55 commented on Oct 3, 2026

    @mossbot55
    Contributor

    Implemented and opened PR #525: #525

    The home hero now carries one muted caption, "Price change over 24h at your current balances; sends and receives aren't included.", rendered only when a priced baseline exists. The token rows are left as they are, so the subject is named once per screen rather than repeated on every row; the caption's plural "figures" covers them.

    Checks: bun run check exit 0 (root 2435 pass, 0 fail; every workspace suite 0 fail); new focused tests for the caption and for its absence without a priced baseline; UI Lab desktop 1280x720 and phone 390x844 passes on the portfolio ready and prices-offline stories.

    One limitation worth naming: no automated accessibility scan of the wallet document is available from the UI Lab bench, so that check could not run here. It is recorded in the PR and tracked as #524.

  3. derek-flossman commented on Oct 13, 2026

    @derek-flossman
    Collaborator

    Fixed in #525, merged into release/v0.10.6 through batch #642 (fecb613).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions