-
Notifications
You must be signed in to change notification settings - Fork 905
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(wallet): Enable Bridging in UI #23817
Conversation
components/brave_wallet_ui/page/screens/swap/assets/lp-icons/lifi.svg
Outdated
Show resolved
Hide resolved
components/brave_wallet_ui/page/screens/swap/assets/lp-icons/lifi.svg
Outdated
Show resolved
Hide resolved
A Storybook has been deployed to preview UI for the latest push |
dc647cc
to
f872667
Compare
A Storybook has been deployed to preview UI for the latest push |
f872667
to
1f35944
Compare
A Storybook has been deployed to preview UI for the latest push |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
strings++
1f35944
to
fef0705
Compare
A Storybook has been deployed to preview UI for the latest push |
5412f4c
to
3980118
Compare
A Storybook has been deployed to preview UI for the latest push |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
core++
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
frontend ++
3980118
to
8587687
Compare
[puLL-Merge] - brave/brave-core@23817 DescriptionThis PR adds support for cross-chain bridging in the Brave Wallet Swap UI. It introduces a new "Bridge" route and tab for bridging assets between different blockchains. The UI has been updated to accommodate both swapping and bridging, with some shared components between the two flows. ChangesChangesbrave_wallet_constants.h:
nav-options.ts:
container.tsx:
composer_controls.style.ts, composer_controls.tsx:
select_address_button.style.ts, select_address_button.tsx:
select_button.style.ts, select_button.tsx:
shared_composer.style.ts:
from_asset.style.ts, from_asset.tsx, to_asset.style.ts, to_asset.tsx:
select_token_modal.tsx:
virtualized_tokens_list.tsx:
send_screen.tsx:
LpMetadata.ts:
quote-info.style.ts, quote-info.tsx:
swap.utils.ts:
useSwap.ts:
swap.tsx:
locale.ts:
wallet_strings.grdp:
Overall, this is a significant change that adds cross-chain bridging functionality to the Brave Wallet Swap UI. The code changes look reasonable, with efforts made to reuse existing components and styles where possible. Security HotspotsThere don't appear to be any major security vulnerabilities introduced in this change. However, a few areas to double check:
But overall, the security risk for this UI change seems low. The backend bridging functionality would need to be carefully reviewed separately. |
A Storybook has been deployed to preview UI for the latest push |
8587687
to
1e91869
Compare
A Storybook has been deployed to preview UI for the latest push |
components/brave_wallet_ui/page/screens/swap/components/swap/quote-info/quote-info.tsx
Outdated
Show resolved
Hide resolved
components/brave_wallet_ui/page/screens/swap/components/swap/quote-info/quote-info.tsx
Outdated
Show resolved
Hide resolved
components/brave_wallet_ui/page/screens/swap/components/swap/quote-info/quote-info.tsx
Outdated
Show resolved
Hide resolved
components/brave_wallet_ui/page/screens/swap/components/swap/quote-info/quote-info.tsx
Outdated
Show resolved
Hide resolved
components/brave_wallet_ui/page/screens/swap/components/swap/quote-info/quote-info.tsx
Outdated
Show resolved
Hide resolved
1e91869
to
598edeb
Compare
598edeb
to
0c20184
Compare
A Storybook has been deployed to preview UI for the latest push |
0c20184
to
9e3fdb5
Compare
A Storybook has been deployed to preview UI for the latest push |
Description
Bridging
withLi.Fi
in theWallet
!Send
,Swap
andBridge
UI stylingComposer UI
nav controlsFollow up PR's for more design and support after this lands:
Resolves brave/brave-browser#38527
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Wallet
and navigate to theBridge
screen.Token
to bridgefrom
.Token
to bridgeto
. note:(Thefrom
tokensSymbol
should be pre-filled in the search bar when selecting a token to bridgeto
.)Amount
to get aquote
.Recipient
button and change theReceiving
accountBridge
transaction. (RecommendBNBChain
toPolygon
for a cheaper transaction for testing)Screen.Recording.mov