Skip to content

fix(predict): align homepage BTC row title font weight - #34523

Draft
jasonculbertson wants to merge 1 commit into
MetaMask:mainfrom
jasonculbertson:fix/predict_homepage-btc-row-title-font
Draft

fix(predict): align homepage BTC row title font weight#34523
jasonculbertson wants to merge 1 commit into
MetaMask:mainfrom
jasonculbertson:fix/predict_homepage-btc-row-title-font

Conversation

@jasonculbertson

Copy link
Copy Markdown

Description

The homepage Predictions discovery BTC live row title rendered with a different typeface/weight than the championship market rows below it. BtcLiveRow used default BodyMd (Regular / Geist-Regular), while PredictMarketRowItem titles relied on style={tw.style('font-medium')}, which sets CSS fontWeight instead of switching to the Geist-Medium font family registered via Expo.

This PR sets both titles to fontWeight={FontWeight.Medium} so they resolve to font-default-medium / Geist-Medium consistently through the design-system Text component.

Changelog

CHANGELOG entry: Fixed inconsistent title font weight on the homepage Predictions BTC live row

Related issues

Refs: Homepage Predictions discovery typography consistency

Manual testing steps

Feature: Homepage Predictions discovery title typography

  Scenario: BTC live row title matches championship market row titles
    Given the user is on the Home tab
    And the Predictions discovery section is visible with the BTC live row and championship market rows

    When the user compares the BTC row title with the EPL/NBA market row titles
    Then both titles use the same Geist Medium body style
    And the BTC row still shows price, price-to-beat, and the Live countdown pill

Screenshots/Recordings

Before

BTC live row title used Regular weight / mismatched typeface vs championship market row titles below it.

After

BTC live row and championship market row titles both use FontWeight.Medium (Geist-Medium). Please attach Before/After simulator screenshots before marking Ready for review.

Pre-merge author checklist

Performance checks (if applicable)

  • I've tested on Android
    • Ideally on a mid-range device; emulator is acceptable
  • I've tested with a power user scenario
    • Use these power-user SRPs to import wallets with many accounts and tokens
  • I've instrumented key operations with Sentry traces for production performance metrics

For performance guidelines and tooling, see the Performance Guide.

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.

Made with Cursor

Use FontWeight.Medium on the BTC live row and market row titles so both
resolve to Geist-Medium via the design-system Text component.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jasonculbertson jasonculbertson self-assigned this Aug 9, 2026
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@jasonculbertson

Copy link
Copy Markdown
Author

Before / After evidence (from local simulator while validating this fix)

Before: BTC live row title weight/typeface did not match the championship market rows beneath it.

After: both titles use FontWeight.Medium via the design-system Text API.

Attach formal cropped Before/After assets in the PR template Screenshots section before marking Ready for review if needed.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant