Skip to content

feat: add ability to overridesRadius for MyLedger app icon usecase#65

Merged
gre-ledger merged 3 commits into
mainfrom
support/overridesRadius
Dec 2, 2025
Merged

feat: add ability to overridesRadius for MyLedger app icon usecase#65
gre-ledger merged 3 commits into
mainfrom
support/overridesRadius

Conversation

@gre-ledger

Copy link
Copy Markdown
Contributor

Adds support for custom border radius on crypto icons, allowing square-rounded icons instead of the default circular crop.

This is necessary to have a rounded square icon version for the Apps list in MyLedger of Ledger Wallet.

Changes

  • Added overridesRadius?: string prop to CryptoIconProps (React)
  • Added overridesRadius?: number prop to CryptoIconNativeProps (React Native)
  • Network icons remain circular (not affected by overridesRadius)
  • Updated Storybook with square icons example (16px radius)
  • Added tests for overridesRadius functionality

Usage

<CryptoIcon 
  ledgerId="bitcoin" 
  ticker="BTC" 
  overridesRadius="16px" 
/>

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Dec 1, 2025

Copy link
Copy Markdown

Deploying crypto-icons-storybook with  Cloudflare Pages  Cloudflare Pages

Latest commit: be93f81
Status: ✅  Deploy successful!
Preview URL: https://b5e17af3.crypto-icons-storybook.pages.dev
Branch Preview URL: https://support-overridesradius.crypto-icons-storybook.pages.dev

View logs

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Dec 1, 2025

Copy link
Copy Markdown

Deploying crypto-icons-storybook-react-native with  Cloudflare Pages  Cloudflare Pages

Latest commit: be93f81
Status: ✅  Deploy successful!
Preview URL: https://650e7e83.crypto-icons-storybook-react-native.pages.dev
Branch Preview URL: https://support-overridesradius.crypto-icons-storybook-react-native.pages.dev

View logs

@gre-ledger gre-ledger force-pushed the support/overridesRadius branch from 9fbf321 to a58b2a4 Compare December 1, 2025 18:03
Comment thread lib/__stories__/CryptoIconList.stories.tsx Outdated
Comment thread lib/__stories__/CryptoIconList.stories.tsx Outdated
Comment thread lib/__stories__/CryptoIconList.stories.tsx Outdated
Co-authored-by: Lucas Werey <73439207+LucasWerey@users.noreply.github.com>
@gre-ledger gre-ledger force-pushed the support/overridesRadius branch from db1cf2e to be93f81 Compare December 1, 2025 18:12
@gre-ledger gre-ledger merged commit 27e85b9 into main Dec 2, 2025
11 checks passed
@gre-ledger gre-ledger deleted the support/overridesRadius branch December 2, 2025 09:25
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.

2 participants