Skip to content

fix: allow wrapping in footer container#683

Merged
serhalp merged 2 commits intonpmx-dev:mainfrom
wojtekmaj:footer-wrap
Feb 2, 2026
Merged

fix: allow wrapping in footer container#683
serhalp merged 2 commits intonpmx-dev:mainfrom
wojtekmaj:footer-wrap

Conversation

@wojtekmaj
Copy link
Copy Markdown
Contributor

Before:

localhost_3000_settings

After:

localhost_3000_settings (1)

Copilot AI review requested due to automatic review settings February 1, 2026 20:57
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Feb 2, 2026 3:40am
npmx.dev Ready Ready Preview, Comment Feb 2, 2026 3:40am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Feb 2, 2026 3:40am

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a footer layout issue by allowing content to wrap when there isn't enough horizontal space. The change adds the sm:flex-wrap Tailwind utility class to prevent overflow on smaller screens above the mobile breakpoint.

Changes:

  • Added sm:flex-wrap class to the footer's flex container to allow wrapping on small screens and larger

Copy link
Copy Markdown
Member

@serhalp serhalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks great, thank you!

Copy link
Copy Markdown
Member

@serhalp serhalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks great, thank you!

@serhalp serhalp enabled auto-merge February 2, 2026 03:38
@serhalp serhalp added this pull request to the merge queue Feb 2, 2026
Merged via the queue into npmx-dev:main with commit 22169b2 Feb 2, 2026
13 checks passed
taskylizard pushed a commit to taskylizard/npmx.dev that referenced this pull request Feb 7, 2026
Co-authored-by: Philippe Serhal <philippe.serhal@gmail.com>
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.

4 participants