Page MenuHomePhabricator

Change in-article images to be full width on phones and larger thumb size on tablets
Closed, DuplicatePublic

Description

As a reader, I want to see larger images so that I can learn better through visuals along with text.

Change in-article images and thumbnails to be larger.
Phone: Full width of text column.
Tablet: Width of 400 pixels. (If image is at least 200 px)
Text wrapping behavior remains the same

Task completed on android app https://phabricator.wikimedia.org/T94646

Event Timeline

KHammerstein raised the priority of this task from to Needs Triage.
KHammerstein updated the task description. (Show Details)
KHammerstein added a subscriber: KHammerstein.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 24 2015, 11:20 PM
KHammerstein renamed this task from Change in-article images to be full width on phones and have a min width on tablets to Change in-article images to be full width on phones and larger thumb size on tablets.Apr 24 2015, 11:21 PM
KHammerstein updated the task description. (Show Details)
KHammerstein set Security to None.
KHammerstein updated the task description. (Show Details)

We need to decide how this will effect the page load times. We need to check whether images are small because they are styled that way or because we are loading images that are small.

KLans_WMF added subscribers: bearND, Dbrant, KLans_WMF.EditedApr 29 2015, 8:59 PM

@bearND @Dbrant @Mholloway any insights that you can share on this would be welcome

kaldari added a subscriber: kaldari.EditedMay 6 2015, 6:56 PM

Let's see what happened with the RFC about pregenerating thumbnails of standard sizes: https://www.mediawiki.org/wiki/Requests_for_comment/Standardized_thumbnails_sizes

Results of spike investigation are at https://phabricator.wikimedia.org/T98387#1286815.

tl;dr: This feature will negatively affect performance and increase bandwidth usage, although there are steps we can take to mitigate some of those effects.