💄 app: implement app theme for base - #1112
Conversation
🦋 Changeset detectedLatest commit: 2dccc03 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughThis PR adds Base-specific theme tokens, chain-aware theme selection, branded card/home rendering, and updated swap token filtering and empty-state handling with new localized copy. ChangesBase-aware app theming
Swap token selection
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Warning Gemini encountered an error creating the review. You can try again by commenting |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: c6706412-2a35-414b-8ab3-59a13238aaf9
⛔ Files ignored due to path filters (2)
src/assets/images/card-bg-base.svgis excluded by!**/*.svgsrc/assets/images/card-signature-base.svgis excluded by!**/*.svg
📒 Files selected for processing (23)
.changeset/clever-crews-pay.md.changeset/true-bottles-cough.mdapp.config.tscommon/wagmi.config.tssrc/app/+html.tsxsrc/app/_layout.tsxsrc/assets/images/types.d.tssrc/components/card/CardDetails.tsxsrc/components/card/exa-card/CardContents.tsxsrc/components/card/exa-card/ExaCard.tsxsrc/components/context/ThemeProvider.tsxsrc/components/home/CardStatus.tsxsrc/components/home/VisaSignatureBanner.tsxsrc/components/home/VisaSignatureSheet.tsxsrc/components/shared/Skeleton.tsxsrc/components/swaps/SelectorModal.tsxsrc/components/swaps/Swaps.tsxsrc/i18n/es-AR.jsonsrc/i18n/es.jsonsrc/i18n/pt.jsonsrc/utils/lifi.tssvgr.config.cjstamagui.config.ts
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b75aa89e5d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
|
/gemini review |
|
Warning Gemini encountered an error creating the review. You can try again by commenting |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 8bd2b528-af9d-4e78-8993-05c463a283fd
📒 Files selected for processing (18)
.changeset/clever-crews-pay.mdcommon/wagmi.config.tssrc/app/+html.tsxsrc/app/_layout.tsxsrc/assets/images/types.d.tssrc/components/card/CardDetails.tsxsrc/components/card/exa-card/CardContents.tsxsrc/components/card/exa-card/ExaCard.tsxsrc/components/context/ThemeProvider.tsxsrc/components/home/CardStatus.tsxsrc/components/home/VisaSignatureBanner.tsxsrc/components/home/VisaSignatureSheet.tsxsrc/components/shared/Skeleton.tsxsrc/i18n/es-AR.jsonsrc/i18n/es.jsonsrc/i18n/pt.jsonsvgr.config.cjstamagui.config.ts
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b9fb2ba6c6
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
Actionable comments posted: 2
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 87cf8455-f61b-46bb-b008-36c6d6bc185f
⛔ Files ignored due to path filters (2)
src/assets/images/card-bg-base.svgis excluded by!**/*.svgsrc/assets/images/card-signature-base.svgis excluded by!**/*.svg
📒 Files selected for processing (22)
.changeset/clever-crews-pay.md.changeset/true-bottles-cough.mdcommon/wagmi.config.tssrc/app/+html.tsxsrc/app/_layout.tsxsrc/assets/images/types.d.tssrc/components/card/CardDetails.tsxsrc/components/card/exa-card/CardContents.tsxsrc/components/card/exa-card/ExaCard.tsxsrc/components/context/ThemeProvider.tsxsrc/components/home/CardStatus.tsxsrc/components/home/VisaSignatureBanner.tsxsrc/components/home/VisaSignatureSheet.tsxsrc/components/shared/Skeleton.tsxsrc/components/swaps/SelectorModal.tsxsrc/components/swaps/Swaps.tsxsrc/i18n/es-AR.jsonsrc/i18n/es.jsonsrc/i18n/pt.jsonsrc/utils/lifi.tssvgr.config.cjstamagui.config.ts
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7371e16b30
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
♻️ Duplicate comments (2)
src/components/shared/Skeleton.tsx (1)
7-8: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winHandle compound Tamagui theme names here.
Line 8 only treats the literal
"dark"as dark.useThemeName()can resolve compound names, so nested dark themes will fall back to the light skeleton variant.src/components/card/CardDetails.tsx (1)
36-41: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winNormalize the Tamagui theme name before branching on light vs dark.
Lines 36-41 switch this component to
useThemeName(), but the later logo branches still rely on exact"light"checks. Compound names like"dark_*"will take the light-logo path.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: af8985a7-e963-4b5a-aec6-c2741157ad2e
⛔ Files ignored due to path filters (5)
src/assets/images/card/card-bg-base.svgis excluded by!**/*.svgsrc/assets/images/card/card-bg.svgis excluded by!**/*.svgsrc/assets/images/card/card-signature-base.svgis excluded by!**/*.svgsrc/assets/images/card/card-signature.svgis excluded by!**/*.svgsrc/assets/images/card/card.svgis excluded by!**/*.svg
📒 Files selected for processing (24)
.changeset/clever-crews-pay.md.changeset/true-bottles-cough.mdcommon/wagmi.config.tssrc/app/+html.tsxsrc/app/_layout.tsxsrc/assets/images/card/svgr.config.mjssrc/assets/images/types.d.tssrc/components/card/CardDetails.tsxsrc/components/card/exa-card/CardContents.tsxsrc/components/card/exa-card/ExaCard.tsxsrc/components/context/ThemeProvider.tsxsrc/components/home/CardStatus.tsxsrc/components/home/VisaSignatureBanner.tsxsrc/components/home/VisaSignatureSheet.tsxsrc/components/shared/Skeleton.tsxsrc/components/swaps/SelectorModal.tsxsrc/components/swaps/Swaps.tsxsrc/i18n/es-AR.jsonsrc/i18n/es.jsonsrc/i18n/pt.jsonsrc/utils/lifi.tssvgr.config.mjstamagui.config.tstsconfig.json
feee0a6 to
1411a0b
Compare
2dd26d4 to
06597bf
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 06597bf67f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cab96e2d63
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
b445eb6 to
3b8e0cb
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 039b8f9d90
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cd1fd34a1f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a5abae3246
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 338f6fef0a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 72658f6b58
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2290865a54
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| for (const [key, { ids, error }] of failures) { | ||
| reportError(new Error(`balances failed for chains ${ids.join(", ")}: ${key}`, { cause: error })); | ||
| const failure = new Error(`balances failed for chains ${ids.join(", ")}: ${key}`, { cause: error }); | ||
| if (ids.includes(chain.id)) throw failure; |
There was a problem hiding this comment.
When the Alchemy balance request for the current destination chain fails but other source-chain balance requests succeed, this shared helper now throws before returning any of those cross-chain balances. getBridgeSources() also uses balancesOptions(account), so the add-funds bridge source list can lose all other-chain assets and block bridging even though the failed current-chain balance is only needed for same-chain/destination display; keep surfacing this error to the swap empty state without making the shared bridge-source query fail.
Useful? React with 👍 / 👎.
1 participant
closes #1044
Summary by CodeRabbit
New Features
Bug Fixes