Page MenuHomePhabricator

RfC: Reducing image quality for mobile
Closed, ResolvedPublic

Description

Task to follow the progress of the RfC Reducing image quality for mobile.

Details

Reference
fl539

Event Timeline

• flimport raised the priority of this task from to Medium.Sep 12 2014, 1:45 AM
• flimport added a project: Architecture.
• flimport set Reference to fl539.
daniel subscribed.

Discussed on 2014-04-16, (partially?) approved.

Minutes from https://www.mediawiki.org/wiki/Architecture_meetings/RFC_review_2014-04-16

  • ''LINK:'' https://www.mediawiki.org/wiki/Requests_for_comment/Reducing_image_quality_for_mobile (sumanah, 21:03:54)
  • I asked Yuri what he wanted: 1) an ok from ops to increase thumbnail storage by 2-3% and number of files by 15%, 2) from core/tim/etc to proceed with the proposed patch <yurik> assuming my proposed path is satisfactory to everyone's involved (sumanah, 21:04:15)
  • ''LINK:'' https://gerrit.wikimedia.org/r/#/c/119661/ Gerrit changeset, "Allow mobile to reduce image quality" (sumanah, 21:09:33)
  • comments were provided on the image quality gerrit patch (TimStarling, 21:42:33)
  • ''LINK:'' https://www.mediawiki.org/wiki/Talk:Requests_for_comment/Reducing_image_quality_for_mobile#File_insertion_syntax on wikitext addition (sumanah, 21:42:56)
  • image scaler backend relatively uncontroversial -- HTML/URL manipulation to access that API is more complex (TimStarling, 21:43:27)
  • gwicke predictably favours Node.JS service (TimStarling, 21:44:42)
  • <yurik> ok, all settled, will implement the first step (core patch), and start implementing JS magic (sumanah, 21:48:52)
  • required modifications: use string instead of integer "qlow-100px-image.jpg", make it JPG only (no png) (yurik, 21:50:31)
  • Tim skeptical about client-side JS rewrite: potential for CPU usage, flicker, image load aborts, browser incompatibilities, etc. (TimStarling, 21:54:34)