-
Notifications
You must be signed in to change notification settings - Fork 567
Dashboard: Migrate contract/account from chakra to tailwind #7741
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
Dashboard: Migrate contract/account from chakra to tailwind #7741
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7741 +/- ##
=======================================
Coverage 56.42% 56.42%
=======================================
Files 904 904
Lines 58665 58665
Branches 4135 4135
=======================================
Hits 33100 33100
Misses 25460 25460
Partials 105 105
🚀 New features to boost your workflow:
|
size-limit report 📦
|
c8b58f6
to
f119804
Compare
f5dce62
to
850b493
Compare
Merge activity
|
<!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on enhancing the UI and refactoring components in the dashboard, specifically in the `account` section. It improves the display of error messages, balances, and NFT ownership, while also updating component structures for better readability and maintainability. ### Detailed summary - Updated error messages in `nfts-owned.tsx` with improved styling. - Refactored `AccountBalance` to use props directly and changed layout to a grid. - Simplified the `AccountPage` component, enhancing readability and structure. - Refactored `DepositNative` to streamline props and improve input handling. - Enhanced `AccountSigner` layout for better visual presentation and clarity. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
f119804
to
5e883ac
Compare
850b493
to
542360c
Compare
PR-Codex overview
This PR focuses on improving the UI and code structure of the account-related components in the dashboard, enhancing readability, and updating styles for better user experience.
Detailed summary
nfts-owned.tsx
with new styles.AccountBalance
component to useStatCard
for displaying balances.AccountPage
to destructure props for better readability.DepositNative
component to improve transaction handling and UI.AccountSigner
layout and styling for better presentation of signer information.