REST API `mobile-html` end-point thumbnails resolution are too big
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	Kelson
	Oct 29 2023, 2:55 PM

Description

Look for example at:
https://bm.wikipedia.org/api/rest_v1/page/mobile-html/Bamak%C9%94

Images:

First (real) image "Bamako_et_fleuve_Niger.jpg" width is 320px. It was 250px with the old mobile-section API
Second (real) image "Bamako_bridge2.jpg" width is 640px. It was 250px with the old mobilde-section API

If we look to the wiki code it is:

[[Fichier:Bamako_et_fleuve_Niger.jpg|thumb|250px|Bamako]]
[[Fichier:Bamako bridge2.jpg|thumb|250px| Bamako Pont, mi bɛ Niger baw kan]]

The images size should:

Respect the wiki code
Be mobile friendly

This bug has a very serious impact on MWoffliner and ZIM snapshots for Kiwix: the team has implemented the support of mobile-html but the ZIM snapshots are far too big to be distributed.

MWoffliner at ticket https://github.com/openzim/mwoffliner/issues/1925

Details

	Subject	Repo	Branch	Lines +/-
	Preserve original src data attribute on lazy load transformation	mediawiki/services/mobileapps	master	+6 -4
	Store original image src as data attribute	mediawiki/services/mobileapps	master	+4 -0

Customize query in gerrit

Related Objects

Mentioned In: T264671: Deprecate and remove MCS endpoints from RESTBase
T348529: Add ability to change image size per request in page/mobile-html endpoint

Event Timeline

Kelson created this task.Oct 29 2023, 2:55 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 29 2023, 2:55 PM

Kelson moved this task from TRIAGE to TOP on the affects-Kiwix-and-openZIM board.Oct 29 2023, 2:55 PM

Kelson mentioned this in T348529: Add ability to change image size per request in page/mobile-html endpoint.

Kelson mentioned this in T264671: Deprecate and remove MCS endpoints from RESTBase.Oct 29 2023, 2:59 PM

MSantos subscribed.Nov 7 2023, 3:39 PM

MSantos added projects: Page Content Service, Content-Transform-Team.Nov 10 2023, 8:05 PM

Kelson updated the task description. (Show Details)Nov 11 2023, 6:50 PM

Kelson updated the task description. (Show Details)

MSantos edited projects, added Mobile-Content-Service, Content-Transform-Team-WIP; removed Content-Transform-Team.Nov 23 2023, 3:14 PM

MSantos added a subscriber: Seddon.

Kelson updated the task description. (Show Details)Nov 23 2023, 5:41 PM

MSantos added a project: Essential-Work.Dec 4 2023, 3:47 PM

For reference, here is the transformation that changes the image resolution that page/html returned.

https://github.com/wikimedia/mediawiki-services-mobileapps/blob/b57d7ea182a4ff8a9fd1a2d7f7f69ebba19d1c7a/lib/mobile/MobileHTML.js#L963

Change #1016333 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/services/mobileapps@master] Store original image src as data attribute

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

gerritbot added a project: Patch-For-Review.Apr 2 2024, 12:27 PM

Change #1016333 merged by jenkins-bot:

[mediawiki/services/mobileapps@master] Store original image src as data attribute

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

Change #1020274 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/services/mobileapps@master] Preserve original src data attribute on lazy load transformation

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

Change #1020274 merged by jenkins-bot: