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

AMuigai triaged this task as Medium priority.Mar 25 2020, 5:53 AM
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.