Skip to content

Reduce footer spacing on mobile#654

Merged
yamcodes merged 4 commits intomainfrom
copilot/fix-footer-spacing-mobile
Dec 27, 2025
Merged

Reduce footer spacing on mobile#654
yamcodes merged 4 commits intomainfrom
copilot/fix-footer-spacing-mobile

Conversation

Copy link
Contributor

Copilot AI commented Dec 27, 2025

The homepage footer had excessive vertical spacing on mobile devices, particularly the top margin, internal padding, and spacing around the HR divider line.

Changes

  • Top margin: mt-32mt-8 md:mt-32 (128px → 32px on mobile, 128px on desktop)
  • Vertical padding: py-14py-8 md:py-14 (56px → 32px on mobile, 56px on desktop)
  • HR line spacing: px-6px-3 md:px-6 (24px → 12px on mobile, 24px on desktop)

Applied to the footer wrapper and content container in apps/www/app/(home)/page.tsx.

Screenshots

Mobile (375px width)

Desktop view unchanged.

Original prompt

This section details on the original issue you should resolve

<issue_title>Footer is too spacious on mobile</issue_title>
<issue_description>On mobile, the footer is too spacious. The issues specifically is the space around the footer, and the very big margin top it has that should be significantly reduced on mobile.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Summary by CodeRabbit

  • Style
    • Optimized footer spacing for improved layout on mobile and tablet devices, reducing top margin and vertical padding on smaller screens while maintaining generous spacing on larger displays.

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

@changeset-bot
Copy link

changeset-bot bot commented Dec 27, 2025

⚠️ No Changeset found

Latest commit: 893a0ea

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Dec 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
arkenv Ready Ready Preview, Comment Dec 27, 2025 3:44pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 27, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

The homepage footer spacing has been adjusted to reduce excessive whitespace on mobile devices. Top margin changed from mt-32 to mt-16 with md:mt-32, and vertical padding reduced from py-14 to py-8 with md:py-14, maintaining larger spacing on medium+ screens.

Changes

Cohort / File(s) Summary
Footer spacing refinement
apps/www/app/(home)/page.tsx
Responsive spacing adjustments: outer wrapper top margin (mt-16md:mt-32) and inner content padding (py-8md:py-14) to reduce mobile footprint while preserving desktop appearance

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • #650: Modifies footer JSX in the same file; adds new footer section while this PR refines spacing
  • #639: Also adjusts responsive layouts in the homepage file; broader hero/header/mobile refinements

Poem

🐰 The footer hops with lighter grace,
No longer sprawling mobile space,
With margins tight and padding wise,
A nimble design now satisfies!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Reduce footer spacing on mobile' accurately describes the main change in the changeset, which reduces footer spacing specifically on mobile devices.
Linked Issues check ✅ Passed The code changes directly address the requirements in issue #653 by reducing top margin and vertical padding on mobile while preserving desktop spacing.
Out of Scope Changes check ✅ Passed All changes are focused on the footer spacing adjustments in apps/www/app/(home)/page.tsx, directly aligned with the linked issue objectives.

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

Co-authored-by: yamcodes <2014360+yamcodes@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix excessive spacing in mobile footer Reduce footer spacing on mobile Dec 27, 2025
Copilot AI requested a review from yamcodes December 27, 2025 15:10
@yamcodes yamcodes marked this pull request as ready for review December 27, 2025 15:21
@github-actions github-actions bot added the www Improvements or additions to arkenv.js.org label Dec 27, 2025
@yamcodes
Copy link
Owner

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 27, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@yamcodes
Copy link
Owner

@copilot almost there, but i want even a bit less space around the hr line in the footer. also, for the space above the footer, even mt-16 is too much. let's try mt-8.

Co-authored-by: yamcodes <2014360+yamcodes@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 27, 2025

@copilot almost there, but i want even a bit less space around the hr line in the footer. also, for the space above the footer, even mt-16 is too much. let's try mt-8.

Done! Changed mt-16 to mt-8 and reduced the HR line spacing from px-6 to px-3 md:px-6 on mobile. (commit 2fca3bf)

Mobile view (375px):

Mobile Footer Updated

The footer now has:

  • Top margin: 2rem (32px) on mobile vs 8rem on desktop
  • HR icon spacing: 0.75rem (12px) on mobile vs 1.5rem on desktop

@yamcodes yamcodes merged commit a7f75ad into main Dec 27, 2025
12 of 13 checks passed
@yamcodes yamcodes deleted the copilot/fix-footer-spacing-mobile branch December 27, 2025 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

www Improvements or additions to arkenv.js.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Footer is too spacious on mobile

2 participants