-
Notifications
You must be signed in to change notification settings - Fork 89
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
[SWAP REFACTORING #4.1] new swap page, setup layout #1043
[SWAP REFACTORING #4.1] new swap page, setup layout #1043
Conversation
Important: example how to customize components - DestCurrencyInputPanel
src/components/swap/SwapHeader.tsx
Outdated
export default function SwapHeader({ allowedSlippage }: { allowedSlippage: Percent }) { | ||
export default function SwapHeader({ | ||
allowedSlippage, | ||
className = '', |
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.
Curious why we use empty string as the default when className
is optional and can be undefined
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.
Fixed
src/custom/pages/NewSwap/index.tsx
Outdated
const { allowedSlippage } = useDerivedSwapInfo() | ||
|
||
return ( | ||
<styledEl.Container> |
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.
shouldn't the swap widget composition be inside a component instead of the page?
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.
I moved it because AffiliateStatusCheck
component is out of boundaries
[SWAP REFACTORING #5] code review fixes
[SWAP REFACTORING #4.5] new swap page, cosmos fixtures and documentation
[SWAP REFACTORING #4.4] new swap page, merge develop, code cleanup
[SWAP REFACTORING #4.3] new swap page, recipient controls, trade rates, high slippage warning
[SWAP REFACTORING #4.2] new swap page, bind data rendering and logic
Summary
https://docs.google.com/document/d/1QCuT2zT-G7fKB5azLmssj8O3yd2dwBpDuoRJuz0cw0U/edit#
How to review
Sorry for the big PR, for easier code review, please review per commit, starting with the oldest
Refactoring PR's list:
4.1. Setup layout
4.2. Bind data rendering and logic
4.3. Recipient controls, trade rates, high slippage warning
4.4. Merge develop, code cleanup
4.5. Cosmos fixtures and documentation
Testing
Don't test this PR, test the final PR - New swap page