Merged
Conversation
Contributor
|
@Roaring30s is attempting to deploy a commit to the Livepeer Foundation Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes a UI collision issue in the transaction confirmation modal where the exit icon overlapped with the success indicator. The changes improve the visual hierarchy and layout of the modal by centering the success badge, removing redundant text, and adding proper spacing.
Changes:
- Removed redundant "Confirmed" text from modal header and centered the enlarged success badge
- Enhanced the success badge with larger icon (22x22) and increased padding for better prominence
- Improved text hierarchy in unbonding transactions by reducing font size of explanatory text
- Added gap spacing to prevent collision between transaction title and receipt link
- Updated .gitignore to exclude TypeScript build artifacts
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| components/TxConfirmedDialog/index.tsx | Enhanced modal UI by centering and enlarging success badge, removing redundant "Confirmed" text, improving text hierarchy in unbonding section, and adding gap to prevent title collision |
| .gitignore | Added TypeScript build artifacts (*.tsbuildinfo and next-env.d.ts) to ignore list |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ECWireless
approved these changes
Feb 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR improves the visual hierarchy and responsive layout of transaction confirmation modals.
Type of Change
Related Issue(s)
Fixes #526
Changes Made
Testing
How to test (optional unless test is not trivial)
Impact / Risk
Risk level: Low
Impacted areas: UI
User impact: Visual
Rollback plan: PR Revert
Screenshots / Recordings (if applicable)
Before
After
Additional Notes
None