You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Two visual bugs were affecting the token price display in the wallet token list (TokenListItemV2) and the explore/trending page.
Bug 1 — Missing thousand-separator commas
The token price in the second row of each list item (e.g. $2283.65 for ETH) was formatted with addCurrencySymbol, which constructs the string via a plain .toString() call and never applies locale-aware number formatting. This meant that prices ≥ $1,000 rendered without commas (e.g. $2283.65 instead of $2,283.65).
Fixed by switching to formatPriceWithSubscriptNotation, the same formatter already used on the asset-overview/price chart page, which uses Intl.NumberFormat('en-US', …) and produces properly comma-separated output.
Bug 2 — Too many decimal places for large prices formatPriceWithSubscriptNotation was configured with maximumFractionDigits: 4 unconditionally, so a price like $2,285.013 (3 significant decimal digits within the allowed 2–4 range) rendered incorrectly. This also affected the explore/trending token row.
Fixed by scoping maximumFractionDigits to the magnitude of the price: 2 decimals for values ≥ 1, 4 decimals for values < 1 (where extra precision matters, e.g. $0.1446). Subscript notation for very small values is unaffected.
Bug 3 — Unintentional bold weight on fiat balance
The fiat balance text variant was BodyMDBold instead of BodyMDMedium, making the balance appear heavier than intended.
The full description has been saved to .agent/chore-remove-bold-and-add-commas-list-item-v2.PR-desc.md.
Changelog
CHANGELOG entry: fixed token prices in the wallet list displaying without thousand-separator commas and with too many decimal places
Feature: my feature nameScenario: user [verb for user action]Given [describe expected initial app state]
When user [verb for user action]
Then [describe expected outcome]
Screenshots/Recordings
Before
After
Look at the title and the USD amount. As you can see it is not bold anymore:
I've completed the PR template to the best of my ability
I've included tests if applicable
I've documented my code using JSDoc format if applicable
I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
Pre-merge reviewer checklist
I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.
Note
Medium Risk
Adjusts a shared price formatter and switches wallet token list rendering to it, which could change how fiat prices appear across multiple screens. Risk is limited to display/formatting (no transaction or balance calculations).
Overview
Fixes token price display in the wallet token list by replacing addCurrencySymbol with formatPriceWithSubscriptNotation, restoring locale-aware formatting (e.g., thousand separators) and consistent currency symbol/suffix handling.
Updates formatPriceWithSubscriptNotation to cap decimals at 2 for values >= 1 while keeping up to 4 decimals (and subscript notation) for smaller values, and adds unit tests to cover the new truncation behavior. Also tweaks the fiat balance text style in TokenListItemV2 from bold to a medium weight.
Written by Cursor Bugbot for commit 18f74fe. This will update automatically on new commits. Configure here.
Behavioral change: For values >= 1, prices are now limited to exactly 2 decimal places instead of up to 4.
This directly affects the Predict (Polymarket) UI where market prices and balances are displayed.
Covered by SmokePredictions (market lists, position lifecycle, balance sync, activities).
TokenListItemV2 change (wallet token list row):
Replaces addCurrencySymbol with formatPriceWithSubscriptNotation for token fiat price display.
Token list is part of the core Wallet screen and impacts asset display formatting across networks.
Token list rendering and fiat display are validated under SmokeWalletPlatform (transaction history, privacy mode, core wallet UI behavior).
Even though this is mostly a visual/formatting change, incorrect formatting could break UI assertions or cause unexpected string mismatches in E2E tests.
No controller, Engine, network, confirmations, or transaction flow logic was changed. No dApp permissions, swaps, perps, or multi-chain session logic affected.
Therefore:
SmokePredictions is required due to direct changes in Predict price formatting utilities.
SmokeWalletPlatform is required due to wallet token list fiat price formatting change.
No need to select SmokeConfirmations, SmokeTrade, SmokeNetwork*, etc., as no transaction or network logic was modified.
Risk is medium because changes affect widely visible wallet UI (token list) and Predict price rendering, but are limited to formatting logic and a presentational component.
Performance Test Selection:
The change only modifies number formatting logic (Intl.NumberFormat maxFractionDigits condition) and a text style variant (BodyMDBold -> BodyMDMedium). No structural UI changes, no new lists, no additional data fetching, no controller/state changes. Performance impact is negligible and does not warrant running performance test suites.
✅ E2E Fixture Validation — Schema is up to date
16 value mismatches detected (expected — fixture represents an existing user). View details
juanmigdr
changed the title
chore: correct token price formatting in wallet token list
chore: correct token price formatting in wallet token list cp-7.69.0 cp7.70.0
Mar 16, 2026
juanmigdr
changed the title
chore: correct token price formatting in wallet token list cp-7.69.0 cp7.70.0
chore: correct token price formatting in wallet token list cp-7.69.1 cp7.70.0
Mar 16, 2026
juanmigdr
changed the title
chore: correct token price formatting in wallet token list cp-7.69.1 cp7.70.0
chore: correct token price formatting in wallet token list cp-7.69.1 cp-7.70.0
Mar 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Two visual bugs were affecting the token price display in the wallet token list (
TokenListItemV2) and the explore/trending page.Bug 1 — Missing thousand-separator commas
The token price in the second row of each list item (e.g.
$2283.65for ETH) was formatted withaddCurrencySymbol, which constructs the string via a plain.toString()call and never applies locale-aware number formatting. This meant that prices ≥ $1,000 rendered without commas (e.g.$2283.65instead of$2,283.65).Fixed by switching to
formatPriceWithSubscriptNotation, the same formatter already used on the asset-overview/price chart page, which usesIntl.NumberFormat('en-US', …)and produces properly comma-separated output.Bug 2 — Too many decimal places for large prices
formatPriceWithSubscriptNotationwas configured withmaximumFractionDigits: 4unconditionally, so a price like$2,285.013(3 significant decimal digits within the allowed 2–4 range) rendered incorrectly. This also affected the explore/trending token row.Fixed by scoping
maximumFractionDigitsto the magnitude of the price: 2 decimals for values ≥ 1, 4 decimals for values < 1 (where extra precision matters, e.g.$0.1446). Subscript notation for very small values is unaffected.Bug 3 — Unintentional bold weight on fiat balance
The fiat balance text variant was
BodyMDBoldinstead ofBodyMDMedium, making the balance appear heavier than intended.The full description has been saved to
.agent/chore-remove-bold-and-add-commas-list-item-v2.PR-desc.md.Changelog
CHANGELOG entry: fixed token prices in the wallet list displaying without thousand-separator commas and with too many decimal places
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/ASSETS-2909 & https://consensyssoftware.atlassian.net/browse/ASSETS-2921
Manual testing steps
Screenshots/Recordings
Before
After
Look at the title and the USD amount. As you can see it is not bold anymore:
Look at the decimals in Ethereum on Mainnet:
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Adjusts a shared price formatter and switches wallet token list rendering to it, which could change how fiat prices appear across multiple screens. Risk is limited to display/formatting (no transaction or balance calculations).
Overview
Fixes token price display in the wallet token list by replacing
addCurrencySymbolwithformatPriceWithSubscriptNotation, restoring locale-aware formatting (e.g., thousand separators) and consistent currency symbol/suffix handling.Updates
formatPriceWithSubscriptNotationto cap decimals at 2 for values >= 1 while keeping up to 4 decimals (and subscript notation) for smaller values, and adds unit tests to cover the new truncation behavior. Also tweaks the fiat balance text style inTokenListItemV2from bold to a medium weight.Written by Cursor Bugbot for commit 18f74fe. This will update automatically on new commits. Configure here.