Skip to content

Commit 2577532

Browse files
CassioMGclaude
andauthored
fix(sign-message): keep the trust warning inside the footer (#2972)
The default View footer is a fixed 80px, which the "Only confirm if you trust this site" line plus the button row overflows by 40px. The text spilled upward out of the footer and painted over the content pane — visible whenever the message is long enough to push the Wallet/Network card to the bottom of the scroll area. Size the footer to its content instead, and add 12px above the warning so it clears the same 36px the footer already clears below the buttons. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 1e8fdfe commit 2577532

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

extension/src/popup/views/SignMessage/index.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,10 @@ export const SignMessage = () => {
359359
</div>
360360
)}
361361
</View.Content>
362-
<View.Footer>
362+
{/* The default footer is a fixed 80px, which this footer's warning line
363+
plus button row overflows — the text then spills up over the content
364+
pane. Size to content so it grows instead. */}
365+
<View.Footer customHeight="auto">
363366
{!shouldShowWarning && (
364367
<span className="SignMessage__Warning">
365368
{t("Only confirm if you trust this site")}

extension/src/popup/views/SignMessage/styles.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,10 @@
6464
}
6565

6666
&__Warning {
67+
// The footer clears 36px below the buttons (24px footer padding + the 12px
68+
// Actions margin). The content pane's own bottom padding contributes 24px
69+
// above this line, so add 12px to match, keeping the line evenly inset.
70+
margin-top: pxToRem(12px);
6771
color: var(--sds-clr-gray-10);
6872
text-align: center;
6973
font-size: pxToRem(14px);

0 commit comments

Comments
 (0)