Skip to content

fix: altpayment overlap footer#607

Open
lissavxo wants to merge 1 commit intomasterfrom
fix/sideshift-overlap-footer
Open

fix: altpayment overlap footer#607
lissavxo wants to merge 1 commit intomasterfrom
fix/sideshift-overlap-footer

Conversation

@lissavxo
Copy link
Collaborator

@lissavxo lissavxo commented Jan 28, 2026

Related to #588

Description

Fix footer hiding when altpayment widget appears.

Test plan

Click in 'Don't have any XEC?' make sure the footer remains there

Summary by CodeRabbit

  • Style
    • Refined the payment widget container's vertical spacing calculations to provide a more balanced and visually optimized layout for improved user experience.
    • Enhanced the footer section styling with additional top padding to ensure improved visual alignment, consistent spacing, and better interface aesthetics.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

📝 Walkthrough

Walkthrough

Two component styling adjustments were made: the SideshiftCtn height calculation in AltpaymentWidget was reduced, and footer top padding was increased in the Widget component. Both changes are CSS-only modifications with no functional logic or interface alterations.

Changes

Cohort / File(s) Summary
AltpaymentWidget height adjustment
react/lib/components/Widget/AltpaymentWidget.tsx
SideshiftCtn container height calculation changed from calc(100% - 20px) to calc(100% - 45px), reducing available vertical space by 25px
Widget footer spacing
react/lib/components/Widget/Widget.tsx
Added paddingTop: '14px' to footer style, increasing top spacing of footer element

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

bug

Suggested reviewers

  • chedieck

Poem

🐰 A hop, a skip, and spacing tweaks so neat,
The footer found its rhythm, now the layout's sweet,
Calculations shifted, height adjusted just right,
These CSS adjustments make the UI bright!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: fixing the footer overlap issue caused by the altpayment widget.
Description check ✅ Passed The description includes all required template sections with adequate detail: related issue (#588), clear non-technical description of the fix, and a specific test plan.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lissavxo lissavxo requested review from Klakurka and chedieck January 28, 2026 22:01
@Klakurka Klakurka added bug Something isn't working enhancement (UI/UX/feature) New feature or request labels Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement (UI/UX/feature) New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants