-
Notifications
You must be signed in to change notification settings - Fork 619
[Dashboard] Fix minor layer issues #4928
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
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4928 +/- ##
=======================================
Coverage 47.68% 47.68%
=======================================
Files 1058 1058
Lines 57290 57290
Branches 3942 3942
=======================================
Hits 27319 27319
Misses 29280 29280
Partials 691 691
*This pull request uses carry forward flags. Click here to find out more. |
size-limit report 📦
|
MananTank
left a comment
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.
Title and All buttons centered looks weird.
Lets keep the title left aligned and all buttons in 2x2 grid - 2 buttons on row1, 2 buttons on row2
MananTank
left a comment
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.
address comments
5493655 to
bebfc15
Compare
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
bebfc15 to
96b41c9
Compare
96b41c9 to
942cc38
Compare
942cc38 to
3fc287d
Compare
Merge activity
|
Before:  After:  <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating the layout and styling of components in the `supply-cards.tsx` and `page.tsx` files to enhance the visual structure and responsiveness of the dashboard interface. ### Detailed summary - In `supply-cards.tsx`, changed the layout from a row (`flex-row`) to a column (`flex-col`) for better alignment. - In `page.tsx`, modified the layout to use `flex-col` with additional styling adjustments, including: - Updated heading class from `font-bold text-xl` to `font-semibold text-2xl tracking-tight`. - Adjusted the container to use a grid layout for better responsiveness. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
3fc287d to
f5b8230
Compare


Before:

After:

PR-Codex overview
This PR focuses on updating the layout and styling of the
supply-cardsandpagecomponents in the NFT dashboard, enhancing responsiveness and visual hierarchy.Detailed summary
supply-cardsfrom a horizontal (flex-row) to a vertical (flex-col) arrangement.page.tsxfromfont-bold text-xltofont-semibold text-2xl tracking-tight.page.tsxcontent to use a grid and modified spacing classes for better alignment.