Skip to content

feat(sra): TokenChainPill component#299

Open
OmarBasem wants to merge 3 commits into
mainfrom
omar/token-select
Open

feat(sra): TokenChainPill component#299
OmarBasem wants to merge 3 commits into
mainfrom
omar/token-select

Conversation

@OmarBasem

@OmarBasem OmarBasem commented Jul 7, 2026

Copy link
Copy Markdown
Member

Closes FS-2384

Summary

This PR implements TokenChainPill component

@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
zerodev-signer-demo Ready Ready Preview, Comment Jul 10, 2026 5:28am

Request Review

@OmarBasem OmarBasem self-assigned this Jul 7, 2026
@OmarBasem OmarBasem changed the title feat: TokenChainPill feat(sra): TokenChainPill Jul 7, 2026
@OmarBasem OmarBasem changed the title feat(sra): TokenChainPill feat(sra): TokenChainPill component Jul 7, 2026
@OmarBasem OmarBasem requested a review from brtkx July 7, 2026 06:59
@OmarBasem OmarBasem force-pushed the omar/token-select branch from 571296d to a928508 Compare July 7, 2026 07:58
@OmarBasem OmarBasem force-pushed the omar/token-select branch from a928508 to 1e612e1 Compare July 8, 2026 15:56
@OmarBasem OmarBasem force-pushed the omar/token-select branch from 376a412 to 78315f9 Compare July 9, 2026 11:17
@OmarBasem OmarBasem changed the base branch from omar/sra-logic to main July 9, 2026 11:41
@OmarBasem OmarBasem force-pushed the omar/token-select branch from ac2a72d to fff48b9 Compare July 9, 2026 11:41
@pkg-pr-new

pkg-pr-new Bot commented Jul 9, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/zerodevapp/zerodev-wallet-sdk/@zerodev/wallet-core@299
npm i https://pkg.pr.new/zerodevapp/zerodev-wallet-sdk/@zerodev/wallet-react@299
npm i https://pkg.pr.new/zerodevapp/zerodev-wallet-sdk/@zerodev/react-ui@299
npm i https://pkg.pr.new/zerodevapp/zerodev-wallet-sdk/@zerodev/wallet-react-ui@299

commit: 7f97fa4

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not part of this PR but once we have UI ready we should also remember to add the same density system we implemented for the wallet.

className?: string
}

export function TokenChainPill({

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this basically a Select primitive with an optional image? Based on its looks and function, it seems like it should be part of react-ui?

@brtkx brtkx Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we did that, we could have this component as Select as well. It just disabled and has chevron hidden.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That component is already included

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this basically a Select primitive with an optional image? Based on its looks and function, it seems like it should be part of react-ui?

Currently it is being used in SRA only. If we needed somewhere else, it can then be refactored to react-ui

# Conflicts:
#	packages/smart-routing-address-react-ui/.storybook/main.ts
#	packages/smart-routing-address-react-ui/.storybook/preview.tsx
#	packages/smart-routing-address-react-ui/package.json
#	packages/smart-routing-address-react-ui/postcss.config.js
#	packages/smart-routing-address-react-ui/src/components/TokenChainPill/TokenChainPill.stories.tsx
#	packages/smart-routing-address-react-ui/src/components/TokenChainPill/TokenChainPill.test.tsx
#	packages/smart-routing-address-react-ui/src/components/TokenChainPill/index.tsx
#	packages/smart-routing-address-react-ui/src/index.ts
#	packages/smart-routing-address-react-ui/src/styles.css
#	pnpm-lock.yaml
# Conflicts:
#	pnpm-lock.yaml
@OmarBasem OmarBasem force-pushed the omar/token-select branch from fff48b9 to 7f97fa4 Compare July 10, 2026 05:26
@OmarBasem OmarBasem requested a review from brtkx July 10, 2026 05:26
@OmarBasem OmarBasem requested review from brtkx and removed request for brtkx July 13, 2026 08:19
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