Fix footer animation cutoff

[Dprof-in-tech]

Description

This pull request makes a minor adjustment to the layout of the Footer component. The minimum height of the footer has been increased for both default and large screen sizes to improve visual spacing.

• Increased the default minimum height from 360px to 440px, and the large screen minimum height from 540px to 640px in the Footer component (app/components/Footer.tsx).

References

fixes the footer animation cutoff directly in prod.

Testing

• This change adds test coverage for new/changed/fixed functionality

Checklist

• I have added documentation and tests for new/changed functionality in this PR
• All active GitHub checks for tests, formatting, and security are passing
• The correct base branch is being used, if not main

By submitting a PR, I agree to Paycrest's Contributor Code of Conduct and Contribution Guide.

Summary by CodeRabbit

• Style
  • Increased footer height on all screen sizes for improved visual spacing and layout balance.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

The Footer component's responsive minimum height classes are increased across breakpoints: from 360px to 440px on mobile, and from 540px to 640px on larger screens. This purely layout-based adjustment expands the footer's vertical space without introducing new functionality.

Changes

Cohort / File(s)	Summary
Footer Styling app/components/Footer.tsx	Updated responsive minimum height classes: min-h-[360px] → min-h-[440px] and lg:min-h-[540px] → lg:min-h-[640px]

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• PR #341 — Applies the identical Footer.tsx height class updates (360→440, 540→640).
• PR #327 — Also modifies the Footer component with a different footer illustration change.
• PR #337 — Updates Footer component animation and illustration styling.

Suggested reviewers

• chibie
• 5ran6

Poem

🐰 A footer grows taller, oh what a sight,
From 360 to 440, reaching new height!
The rabbit hops round, reviewing with care,
Layout adjustments floating in air! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Fix footer animation cutoff' clearly and concisely describes the main purpose of the PR, which is fixing a footer animation cutoff issue through layout adjustments.
Description check	✅ Passed	The PR description covers the purpose, implementation details, and references an issue. However, testing and documentation checklist items remain unchecked despite being required template sections.
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

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4a7a477 and 34f208d.

📒 Files selected for processing (1)

• app/components/Footer.tsx

🔇 Additional comments (1)

app/components/Footer.tsx (1)

121-121: LGTM!

The increased min-height values (440px mobile, 640px large screens) appropriately accommodate the rocket animation, which moves up to 120px vertically and scales up to 1.22x during the pending state. This should resolve the animation cutoff issue.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[5ran6]

LGTM