Skip to content

fix: adjust layout and styling in transfer summary component#806

Merged
jinoosss merged 1 commit intomainfrom
fix-scroll-ui-issue
Mar 2, 2026
Merged

fix: adjust layout and styling in transfer summary component#806
jinoosss merged 1 commit intomainfrom
fix-scroll-ui-issue

Conversation

@jinoosss
Copy link
Copy Markdown
Member

@jinoosss jinoosss commented Mar 2, 2026

What type of PR is this?

  • bug

What this PR does:

  • Add auto-scroll behavior to transfer summary page when a transaction simulate error occurs, ensuring the error banner is visible to the user
  • Set explicit height (calc(100vh - 48px)) on CommonFullContentLayout to properly constrain the layout below the sticky header
  • Add overflow-y: auto to TransferSummaryLayout and scroll to the bottom via ref when simulateErrorMessage is set
  • Adjust bottom spacer height from 96px to 116px and add filled prop to BottomFixedButtonGroup for correct spacing

@jinoosss jinoosss requested a review from a team as a code owner March 2, 2026 08:00
@jinoosss jinoosss requested a review from dongwon8247 March 2, 2026 08:00
@jinoosss jinoosss merged commit 844cca4 into main Mar 2, 2026
3 checks passed
@jinoosss jinoosss deleted the fix-scroll-ui-issue branch March 2, 2026 08:07
@dongwon8247 dongwon8247 mentioned this pull request Mar 19, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dongwon8247 dongwon8247 dongwon8247 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jinoosss @dongwon8247