Page MenuHomePhabricator

Add image - tif file not displayed upon insertion
Closed, ResolvedPublic

Description

  1. On mobile arwiki wmf.17, for the article - https://ar.wikipedia.org/wiki/268_(عدد) - the following image was suggested:

Screen Shot 2022-01-18 at 2.15.49 PM.png (1×686 px, 147 KB)

  1. The full-screen view of the image (clicking on the image in the card) will display a placeholder image:

Screen Shot 2022-01-18 at 2.16.01 PM.png (1×712 px, 33 KB)

  1. Clicking 'Yes' - the image placeholder will be displayed in the article:

Screen Shot 2022-01-18 at 2.16.22 PM.png (1×668 px, 61 KB)

Notes:

  • the fact that file type is .tiff might be not related to the display issue
  • desktop Add image card doesn't display the full-screen view of the image, but inserting seems to be ok.

Event Timeline

The original URL for the tif file was used in the fullscreen view and during insertion since the target widths for those two screens are larger than the image's original width. In this case, the thumbnail URL should be generated regardless since mustRender=true.

Change 755469 had a related patch set uploaded (by MewOphaswongse; author: MewOphaswongse):

[mediawiki/extensions/GrowthExperiments@master] Add an image: always generate thumbnail URL when mustRender=true

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

InspectorImage viewerCaptionSave dialog
Desktop
Screen Shot 2022-01-19 at 10.55.04 AM.png (634×962 px, 123 KB)
Screen Shot 2022-01-19 at 10.55.10 AM.png (1×2 px, 1 MB)
Screen Shot 2022-01-19 at 10.55.17 AM.png (1×682 px, 401 KB)
Screen Shot 2022-01-19 at 11.00.56 AM.png (1×1 px, 239 KB)
Mobile
Screen Shot 2022-01-19 at 11.02.56 AM.png (642×712 px, 114 KB)
Screen Shot 2022-01-19 at 11.03.14 AM.png (1×708 px, 482 KB)
Screen Shot 2022-01-19 at 11.03.36 AM.png (996×678 px, 432 KB)
Screen Shot 2022-01-19 at 11.04.00 AM.png (1×706 px, 206 KB)

Change 755469 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Add an image: always generate thumbnail URL when mustRender=true

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

Checked on arwiki wmf.20 - the screenhots from @mewoph's comment above match the production. Below is an additional screenshot on how the img would look in the article:

Screen Shot 2022-02-03 at 5.20.19 PM.png (1×2 px, 635 KB)