Page MenuHomePhabricator

Card height incorrect when there's an image
Closed, ResolvedPublic1 Estimated Story PointsBUG REPORT

Description

Cards that have an image should be 512px high, but they currently exceed that (I couldn't immediately find such requirement in the designs, so I assume it's an artefact of the old prototype, but I could be wrong)

The image currently has a min-height that prevents it from being shrunk down any further.
Along with the 7 lines of text & attribution information, it simply can't render at 512px height.

Screenshot 2026-05-18 at 15.10.02.png (394×720 px, 185 KB)

Either we need to cut back on the textual content, or allow the image to shrink to whatever it needs in order to keep the entire card at 512px.

Acceptance criteria

  • Card with image has a height of 512px

Event Timeline

matthiasmullie added a subscriber: JScherer-WMF.

@JScherer-WMF Can you pitch in on whether we should allow the image to shrink slightly, or show less text?

Change #1287873 had a related patch set uploaded (by Matthias Mullie; author: Matthias Mullie):

[mediawiki/extensions/ReaderExperiments@master] Allow image to shrink as needed

https://gerrit.wikimedia.org/r/1287873

Change #1287873 merged by jenkins-bot:

[mediawiki/extensions/ReaderExperiments@master] Allow image to shrink as needed

https://gerrit.wikimedia.org/r/1287873

made the right call to crop/shrink the image versus reducing text as per Figma annotations:

The crop gets shorter to accommodate the extra text height.

mfossati subscribed.

Before (screenshot from local env):

Screen Shot 2026-05-19 at 12.03.15.png (1,440×3,040 px, 1 MB)

After (screenshot from prod): ✅

Screen Shot 2026-05-19 at 12.00.50.png (1,440×3,040 px, 1 MB)