Skip to content

fix: prevent avatars from being shrank#559

Merged
danielroe merged 1 commit intonpmx-dev:mainfrom
wojtekmaj:no-shrink
Jan 31, 2026
Merged

fix: prevent avatars from being shrank#559
danielroe merged 1 commit intonpmx-dev:mainfrom
wojtekmaj:no-shrink

Conversation

@wojtekmaj
Copy link
Copy Markdown
Contributor

@wojtekmaj wojtekmaj commented Jan 31, 2026

Fixes #510

Addresses the issue with two fixes:

  • Explicitly disallows shrinking of avatars/org logos
  • Allows flex wrap to prevent elements from shrinking too much

Before:

image

"During" (fix 1):

image

After (both fixes):

image

Copilot AI review requested due to automatic review settings January 31, 2026 21:15
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 31, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jan 31, 2026 9:17pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Jan 31, 2026 9:17pm
docs.npmx.dev Skipped Skipped Jan 31, 2026 9:17pm

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 visual bug where avatars and organization logos were shrinking inappropriately in flex layouts on narrow screens.

Changes:

  • Changed avatar sizing from w-16 h-16 to size-16 (Tailwind shorthand) and added shrink-0 to prevent flex shrinking
  • Added flex-wrap to parent containers to allow content wrapping instead of squeezing elements

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
app/pages/~[username]/orgs.vue Applied flex-wrap and shrink-0 to user organizations page avatar
app/pages/~[username]/index.vue Applied flex-wrap and shrink-0 to user profile page avatar
app/pages/@[org].vue Applied flex-wrap and shrink-0 to organization page logo

@danielroe danielroe added this pull request to the merge queue Jan 31, 2026
Merged via the queue into npmx-dev:main with commit f8ee1c3 Jan 31, 2026
19 of 20 checks passed
@wojtekmaj wojtekmaj deleted the no-shrink branch January 31, 2026 21:53
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.

Avatar shrinking on mobile

3 participants