Skip to content

fix(app-frontend): instances' images don't break library layout - #7319

Open
VoOneOK wants to merge 1 commit into
modrinth:mainfrom
VoOneOK:fix/library-layout-broken-by-images
Open

fix(app-frontend): instances' images don't break library layout#7319
VoOneOK wants to merge 1 commit into
modrinth:mainfrom
VoOneOK:fix/library-layout-broken-by-images

Conversation

@VoOneOK

@VoOneOK VoOneOK commented Aug 26, 2026

Copy link
Copy Markdown

Description

Previously instances with vertical images (height greater than width) would break library grid-like layout.

Solved by adding max-w-full class (max-width: 100%) to image wrapper

Showcase

Before

before

After

after

Previously instances with vertical images (height greater than width) would break library grid-like layout. Solved by adding max-w-full class (max-width: 100%) to image wrapper
@github-actions

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@Hallskii
Hallskii requested a review from tdgao September 4, 2026 16:18

@tdgao tdgao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@tdgao

tdgao commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

@Mergifyio queue

@mergify

mergify Bot commented Sep 4, 2026

Copy link
Copy Markdown

Merge Queue Status

  • 🟠 Waiting for queue conditions
  • ⏳ Enter queue
  • ⏳ Run checks
  • ⏳ Merge
Waiting for any of
  • check-neutral = Lint and Test
  • check-skipped = Lint and Test
  • check-success = Lint and Test
All conditions
  • any of [🔀 queue conditions]:
    • all of [📌 queue conditions of queue rule default]:
      • any of [🛡 GitHub branch protection]:
        • check-neutral = Lint and Test
        • check-skipped = Lint and Test
        • check-success = Lint and Test
      • github-review-approved [🛡 GitHub branch protection]
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of [📌 queue -> configuration change requirements]:
    • -mergify-configuration-changed
    • check-success = @mergify/Configuration changed
    • check-success = @mergify/Configuration has been deleted
  • any of [📌 queue requirement]:
    • check-neutral = @mergify/Mergify Merge Protections
    • check-skipped = @mergify/Mergify Merge Protections
    • check-success = @mergify/Mergify Merge Protections

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants