Page MenuHomePhabricator

Portrait images stretched to fit screen in gallery view on Jio 2
Closed, ResolvedPublic

Description

Issue

On viewing a gallery within an article, portrait images are stretched to fit the screen

How to replicate

On a Jio 2 phone, go to the George Clooney article, and select the image gallery. On the first image, his portrait is stretched to fit the screen.

IMG_5567.jpg (4×3 px, 610 KB)

Request

Support images in portrait mode

Event Timeline

SBisson renamed this task from Portrait images stretched to fit screen to Portrait images stretched to fit screen in gallery view.Mar 25 2020, 10:25 AM
SBisson added a project: Inuka-Team.
AMuigai renamed this task from Portrait images stretched to fit screen in gallery view to Portrait images stretched to fit screen in gallery view on Jio 2.Apr 6 2020, 8:51 PM
AMuigai edited projects, added Inuka-Team (Kanban); removed Inuka-Team.
AMuigai updated the task description. (Show Details)

@Jpita @SGautam_WMF @AMuigai— could someone with a jio2 device please test out George Clooney on branch T248443-images-jio2? This is only a first test, no PR created yet. Extra points if you can include a picture of what you see of that first gallery image to compare with that's on the task description. Thanks!

This comment was removed by Jpita.

@eamedina George Clooney's image is looking good. I did check some other articles(visual arts, The barker, Indian subcontinent, etc.) as well where an image aspect ratio is less than 1 and images are looking good on those articles as well.

When working on this task, a new bug came up with respect to the black background in the gallery. We are opting for separating this bug into a different ticket and leaving the background white for the time being, read more about this on T250642.

This is working as expected, moving it for product signoff.