Skip to content

fix: stick footer to bottom on short pages (closes #58)#59

Merged
O2sa merged 1 commit intoO2sa:mainfrom
Ai-chan-0411:feat/fix-footer-position
Apr 12, 2026
Merged

fix: stick footer to bottom on short pages (closes #58)#59
O2sa merged 1 commit intoO2sa:mainfrom
Ai-chan-0411:feat/fix-footer-position

Conversation

@Ai-chan-0411
Copy link
Copy Markdown
Contributor

Summary

  • Add flex flex-col to <main> so the flex container fills the full viewport height (min-h-screen)
  • Add flex-1 w-full to the main content wrapper div so it grows to push the footer down

This ensures the footer is always pinned to the bottom of the viewport when there is little content, while still scrolling naturally below the content on longer pages.

Closes #58

Test plan

  • Open the app with no comparison loaded — footer should sit at the very bottom of the viewport
  • Run a comparison — footer should appear below the result cards, scrolling into view normally
  • Resize the browser window to a short height — footer should not overlap content

🤖 Generated with Claude Code

Use flex column layout on <main> with min-h-screen so the footer
always stays at the bottom of the viewport. On long pages the footer
scrolls naturally after the content.

- Add flex flex-col to <main className="min-h-screen">
- Add flex-1 w-full to the main content wrapper div

Signed-off-by: Ai-chan-0411 (藍) <aoikabu12@gmail.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 12, 2026

@Ai-chan-0411 is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 12, 2026

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

Project Deployment Actions Updated (UTC)
dev-impact Ready Ready Preview, Comment Apr 12, 2026 0:20am

@O2sa O2sa merged commit fea5f28 into O2sa:main Apr 12, 2026
3 checks passed
@O2sa
Copy link
Copy Markdown
Owner

O2sa commented Apr 12, 2026

After now you can work on the complicated issues only and let the easy ones for the new contributors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix footer position: stick to bottom on short pages, scroll on long pages

2 participants