Skip to content

fix: improve error UI in transaction result#805

Merged
jinoosss merged 4 commits intomainfrom
fix-minor-ui-fix
Mar 1, 2026
Merged

fix: improve error UI in transaction result#805
jinoosss merged 4 commits intomainfrom
fix-minor-ui-fix

Conversation

@jinoosss
Copy link
Copy Markdown
Member

@jinoosss jinoosss commented Mar 1, 2026

What type of PR is this?

  • bug

What this PR does:

Summary

  • Simplify scanner button hover styles in transaction result to use opacity-based feedback instead of color changes
  • Add simulate error banner to Transfer Summary page, separating "Insufficient network fee" (shown in the fee section) from detailed simulation errors (shown in a dedicated error banner), consistent with the Approve Transaction page
  • Standardize all .error-message font sizes from 11px to 13px across 11 style files for better readability

Changes

Transaction Result

  • Replace color-based hover effect with opacity transition on the scanner (Gnoscan) button

Transfer Summary

  • Add simulateErrorBannerMessage prop to display simulation errors in a red banner (ERROR: {message})
  • Separate error cases: NetworkFee only shows "Insufficient network fee" for balance issues; simulate errors are shown exclusively in the banner
  • Exclude simulate errors from isNetworkFeeError to prevent duplicate error indicators
  • Fix bottom content overlap with fixed buttons by using a spacer element instead of padding-bottom

Error Message Font Size

  • Override .error-message font-size to 13px in: NetworkFee, StorageDeposit, ApproveTransaction, ApproveSignedDocument, BroadcastMultisigTransaction, CreateMultisigTransaction, TransferSummary, NFTTransferSummary, AddressInput, MemoInput, CustomNetworkInput

@jinoosss jinoosss requested a review from a team as a code owner March 1, 2026 14:11
@jinoosss jinoosss requested a review from dongwon8247 March 1, 2026 14:11
@jinoosss jinoosss merged commit 3b3e517 into main Mar 1, 2026
2 checks passed
@jinoosss jinoosss deleted the fix-minor-ui-fix branch March 1, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant