-
Notifications
You must be signed in to change notification settings - Fork 603
[MNY-249] Dashboard: /<chain>/tx/<tx-hash> page UI improvements #8234
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 GitHub.
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
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 (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8234 +/- ##
=======================================
Coverage 55.02% 55.02%
=======================================
Files 919 919
Lines 60583 60583
Branches 4127 4127
=======================================
Hits 33335 33335
Misses 27145 27145
Partials 103 103
🚀 New features to boost your workflow:
|
size-limit report 📦
|
Merge activity
|
ef700b4
to
c7ea93d
Compare
<!-- ## 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 `page.tsx` component for displaying transaction details in a more structured and user-friendly manner. It introduces new UI elements and improves data presentation for better clarity. ### Detailed summary - Replaced the header and layout structure with a breadcrumb navigation. - Added a `GridItem` component for better organization of transaction details. - Introduced tooltips for additional information on fields. - Enhanced the transaction status display with icons. - Improved the presentation of transaction input data using `PlainTextCodeBlock`. - Removed unnecessary imports and components for cleaner code. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
c8b0cf1
to
3ac2bf1
Compare
c7ea93d
to
8542ebe
Compare
8542ebe
to
305d0d8
Compare
<!-- ## 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 `page.tsx` component for transaction details in the dashboard, improving its layout and user experience by introducing new components and restructuring sections for better clarity and functionality. ### Detailed summary - Added `Breadcrumb` for navigation. - Replaced static text with `CopyTextButton` for transaction hash, from, and to addresses. - Introduced `GridItem` component for consistent layout of transaction details. - Enhanced transaction details display with tooltips and icons. - Improved formatting of timestamps and values. - Added sections for transaction fee, gas prices, and burnt fees. - Included input data display using `PlainTextCodeBlock`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
305d0d8
to
e74aca9
Compare
458097d
to
85a6afc
Compare
PR-Codex overview
This PR focuses on enhancing the
page.tsx
component for transaction details in the dashboard. It introduces a more structured layout, improved UI elements, and additional features like tooltips and copy buttons for better user interaction.Detailed summary
GridItem
component for consistent data presentation.CopyTextButton
for transaction hash, from, and to addresses.date-fns
.Separator
.