Images are too small in MobileFrontend
Closed, DuplicatePublic
Actions

Assigned To

None

Authored By

	Ragesoss
	Mar 26 2014, 7:29 PM

Description

Larger devices

Normal thumbnails on Nexus 7, portrait mode

On larger devices, especially tablets, images in articles are usually too small.

This is usually even more dramatic in landscape mode, even when the layout switches to having the images and text next to each other.

It's even more dramatic when the images have the 'upright' parameter, which makes the mobile images extra small.

See attached screenshots from a Nexus 5 and a Nexus 7.

Version: unspecified
Severity: enhancement

Attached:

Screenshot_2014-03-26-14-53-20.png (1×1 px, 493 KB)

Galleries

The problem is also highly noticeable on gallery pages for example http://en.m.wikipedia.beta.wmflabs.org/wiki/Gallery

Screen Shot 2016-05-17 at 5.20.11 PM.png (542×310 px, 118 KB)

Details

Reference: bz63134

Related Objects
Search...

Status	Subtype	Assigned	Task
Open	Feature	None	T40891 MediaWiki API 2.0 (tracking)
Resolved		Jdlrobson	T95741 API: Inconsistent response for empty collections
Duplicate		None	T65134 Images are too small in MobileFrontend
Resolved		Jdforrester-WMF	T69703 Change MediaWiki default thumbnail size to 300px based on user behavior
Invalid		Edokter	T67160 Refresh thumbnail style in Vector
Open		None	T69709 Change default thumbnail size to 300px for all WMF wikis
Declined		None	T101046 [EPIC] Use MCS as parser for main content in mobile web
Declined		None	T78734 Support for mobile's section use cases in Parsoid HTML
Resolved		ssastry	T114072 <section> tags for MediaWiki sections
Resolved		• bearND	T178705 Prepare MCS for Parsoid section tags
Resolved		• bearND	T178706 Improve section parsing in mobile-sections endpoint
Resolved		• bearND	T178707 Create script to compare old and new sectioning code
Resolved		• bearND	T178708 Update Wiktionary definition parsing to account for new (possibly nested) section tags
Open		None	T114194 MediaWiki should be able to support multiple parser engines
Resolved		• Jhernandez	T114279 Parsoid API responses contain unnecessary meta data bloating HTML response
Open		None	T78676 Store & load data-mw separately
Resolved		• marcoil	T88290 Update data-mw encoding for <ref> tags to point to the HTML content in the <references /> output rather than duplicating it
Resolved		Catrope	T88650 Support data-mw.body.id for reference contents
Resolved		• marcoil	T88660 Parsoid Cite: Render missing reflists
Resolved		Arlolra	T76518 Errors in Parsoid v2 entry point
Resolved		Arlolra	T54936 Move data-parsoid and possibly data-mw out of the DOM, add uids
Resolved		• GWicke	T50483 Expose a parsoid public web API end point in the MediaWiki API
Resolved		• GWicke	T89481 RESTBase beta release (revision storage / content API)
Resolved		• GWicke	T1228 RESTbase deployment
Resolved		fgiunchedi	T76986 RESTBase production hardware
Resolved		fgiunchedi	T88805 rack and setup restbase production cluster in eqiad
Resolved		• Cmjohnson	T83551 Split memcached in eqiad across multiple racks/rows
Resolved		• Cmjohnson	T82259 mc1016 mgmt not working
Resolved		Joe	T89345 Missing memcached logs
Resolved		fgiunchedi	T89639 restbase1006 faulty disk controller
Resolved		• GWicke	T75955 RESTBase / Parsoid integration - waiting for parsoid deploy
Resolved		• Jdouglas	T88456 On-demand generation of HTML and data-parsoid
Resolved		• GWicke	T88457 html2wt conversion support
Resolved		• GWicke	T88588 Formalize the RESTBase v1 specification
Resolved		Arlolra	T89411 Parsoid v2 API selser might not be working correctly
Resolved		• GWicke	T76370 RESTBase performance testing
Resolved		• mobrovac	T76165 Handle revision deletion and suppression in RESTBase
Resolved		• mobrovac	T87520 Set up update jobs for RESTBase: code done and deployed, to be configured
Resolved		• mobrovac	T88652 Integrate revisions into /page structure. Was: Create a /page/revision/{revision} entry point
Resolved		• mobrovac	T87392 Add property & index on namespace in page_revisions
Duplicate		• Jdouglas	T78214 Documentation location and deployment process
Resolved		Andrew	T88850 Import Cassandra packages for Jessie
Resolved		• GWicke	T89461 trebuchet puppet provider broken on systems without upstart
Resolved		fgiunchedi	T89636 setup internal LVS for restbase eqiad servers
Resolved		fgiunchedi	T78514 Detailed cassandra monitoring: metrics and dashboards done, need to set up alerts
Resolved		fgiunchedi	T93886 Cassandra/CQL query interface monitoring
Resolved		Eevans	T93884 graphs for Cassandra metrics
Resolved		fgiunchedi	T97024 some cassandra metrics sent with invalid values
Resolved		• GWicke	T78194 Public entry point for RESTBase
Resolved		• GWicke	T89567 Load test production cluster
Resolved		fgiunchedi	T89366 Access to restbase / cassandra cluster
Resolved		• mobrovac	T91102 Update / maintain Beta Cluster restbase cluster: Up & working with VE
Resolved		• GWicke	T91188 Need to change data model for >~300 wikis in single cassandra cluster
Resolved		• GWicke	T128231 All known clients of Parsoid HTML (either directly hitting Parsoid or via RESTBase) should provide Accept: headers in their requests
Resolved		• AlexMonk-WMF	T128233 VisualEditor: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		• santhosh	T128235 CX Server: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		ssastry	T128236 Flow: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		• Mholloway	T128237 Mobile Content Service: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		Jackmcbarn	T128238 EPH Gadget: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		Renxiaoyi	T128239 Google: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		cscott	T130685 OCG: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Resolved		Kelson	T130686 Kiwix: Add Accept: header to your HTML requests to RESTBase/Parsoid as per T124365
Open		None	T114413 Support various conversions in Parsoid's pb2pb endpoint
Open		None	T130639 All known clients of Parsoid HTML that require data-mw should fetch data-mw separately (if using RESTBase) or process the data-mw blob in Parsoid's pagebundle API response (if using Parsoid directly)
Open		None	T88623 On Parsoid HTML read pages, VisualEditor should be able to load data-mw from a separate API call and zip it into the DOM
Resolved		Arlolra	T130638 Add data-mw as a separate JSON blob in the pagebundle output of Parsoid's API
Resolved		Arlolra	T91700 Store and unpack data-* attributes when serializing/parsing document fragments throughout the pipeline
Open		None	T130643 Content Translation should load data-mw from a separate API call alongside the body content
Declined		None	T130687 OCG should load data-mw from a separate API call alongside the body content
Open		None	T130689 Google's services should load data-mw from a separate API call to RESTBase
Open		None	T124837 Update Flow for Parsoid changes re data-mw
Declined		None	T125885 Switch Flow storage to store a page bundle blob instead of HTML
Declined		None	T125886 Run migration script (HTML => wikitext => page bundle) in production
Declined		None	T125888 Write migration script (HTML => wikitext => page bundle) in production
Open		None	T125889 Update code (extractors, converters) that uses data-mw to deal with separate components
Open		None	T125890 Figure out how to get page bundle from Flow VE
Invalid		None	T194980 Triage
Invalid		None	T130641 Allocate a storage bucket for Parsoid's data-mw attribute
Duplicate		None	T114282 Red links via Parsoid
Duplicate		None	T114286 Resolving styling inconsistencies between HTML returned by Parsoid and HTML returned by PHP parser
Resolved		• Jhernandez	T113769 Spike: Scope the first version of JavaScript-based prototype on existing Parsoid service
Resolved		Jdlrobson	T113649 Demonstrate impact of loading just lead sections of a group of static pages
Duplicate		Peter	T114216 Support CSV output from the WPT wrapper script
Resolved		• bmansurov	T115073 [6 hours] Test a subset of pages with only a lead section [take 2]
Declined		None	T145219 [EPIC] Cleanup MobileFormatter and MobileViewAPI where they overlap
Duplicate		None	T109461 [EPIC]: Improvements to mobileview API
Resolved		• nray	T86210 MobileFrontend throws away warnings and other data from action=parse
Resolved		phuedx	T69054 API mobileview "protection" sometimes returns object, sometimes returns empty array in JSON output
Resolved		Anomie	T12887 API JSON formatter returns [] as an empty return value - inconsistent with {} for non-empty values
Resolved		Anomie	T76728 Clean up ApiResult and ApiFormatXml, create new formatversion
Resolved		Anomie	T85236 Better unit testing for ApiFormat*
Resolved		EBernhardson	T90745 Flow ApiFlowModerateTopicTest::testModerateTopic() assumes ApiResult will pass through arbitrary objects
Resolved		Anomie	T98083 add checkbox to ApiSandbox to use formatversion=2
Resolved		Anomie	T98086 improve RELEASE-NOTES-1.25 for formatversion=2 change
Open		None	T212069 API action=wbgetentities does not handle formatversion=2
Duplicate		None	T78479 API error messages should be localised
Resolved		None	T98528 mobileview "thumbwidth" parameter doesn't always result in return of desired image url for SVG files
Resolved		phuedx	T156408 Create interface between Skin and MobileFormatter
Declined		None	T146396 Improve performance of mobile references API

Event Timeline

• bzimport raised the priority of this task from to Low.Nov 22 2014, 3:05 AM

• bzimport added a project: MobileFrontend-Feature-requests.

• bzimport set Reference to bz63134.

• bzimport added a subscriber: Unknown Object (MLST).

Ragesoss created this task.Mar 26 2014, 7:29 PM

bingle-admin wrote:

Prioritization and scheduling of this bug is tracked on Mingle card https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1803

Created attachment 14919
Upright thumb image on Nexus 7, portrait mode

Attached:

Screenshot_2014-03-26-15-20-03.png (1×1 px, 396 KB)

Created attachment 14920
Upright thumbnail on Nexus 7, landscape mode

Attached:

Screenshot_2014-03-26-15-22-31.png (1×1 px, 370 KB)

Created attachment 14921
Normal thumbnail on Nexus 5, portrait mode

Attached:

Screenshot_2014-03-26-15-16-46.png (1×1 px, 680 KB)

Created attachment 14922
Normal thumbnails on Nexus 5, landscape mode

Attached:

Screenshot_2014-03-26-15-17-45.png (1×1 px, 434 KB)

I've created a new story card in Mingle to track new work on this as this is an enhancement (https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1824). This is likely something we'll need to tackle before we start automatically redirecting tablets to MobileFrontend.

In the https://en.m.wikipedia.org/wiki/Rittergut_K%C3%BCrbitz article the image at the top is declared in wikitext using:

[[File:Rittergut Kürbitz um 1860.jpg|thumb|The feudal estate, around 1860]]

It seems when no width parameter is given it defaults to 220px. Maybe it should default to something higher? I'm not sure where this is configured though... but not much we can do in MobileFrontend.

Make images on large screens float like on desktop?

Jdlrobson moved this task from Needs triage to In Discussion / Open Questions on the MobileFrontend-Feature-requests board.Dec 9 2014, 9:27 PM

• Jaredzimmerman-WMF added a subtask: T67160: Refresh thumbnail style in Vector.Dec 9 2014, 9:44 PM

• Jaredzimmerman-WMF added a subtask: T69709: Change default thumbnail size to 300px for all WMF wikis.

Jdlrobson merged a task: T97199: Change in-article images to be full width on phones and larger thumb size on tablets.Aug 4 2015, 6:57 PM

Jdlrobson added a project: Web-Team-Backlog.

Jdlrobson set Security to None.

Jdlrobson moved this task from Incoming to 2015-16 Q4 on the Web-Team-Backlog board.

Jdlrobson added subscribers: • Mholloway, • KLans_WMF, Dbrant and 4 others.

Possibly something we could fix if we transitioned to Parsoid...

Jdlrobson edited projects, added MobileFrontend; removed Web-Team-Backlog, MobileFrontend-Feature-requests.Oct 1 2015, 5:39 PM

kaldari unsubscribed.Oct 1 2015, 6:16 PM

Jdlrobson moved this task from Backlog to Team: web on the MobileFrontend board.Nov 4 2015, 5:01 PM

Luke081515 closed subtask T69709: Change default thumbnail size to 300px for all WMF wikis as Declined.Feb 2 2016, 12:22 AM

Luke081515 reopened subtask T69709: Change default thumbnail size to 300px for all WMF wikis as Open.Feb 2 2016, 12:45 AM

MBinder_WMF added a project: Web-Team-Backlog.Apr 27 2016, 4:22 PM

Jdlrobson moved this task from 2015-16 Q4 to Incoming on the Web-Team-Backlog board.Apr 27 2016, 4:36 PM

• ggellerman moved this task from Incoming to Needs Prioritization on the Web-Team-Backlog board.May 11 2016, 4:51 PM

@dr0ptp4kt @JKatzWMF interested?

• Jhernandez added a project: Design.May 11 2016, 4:52 PM

I think it's okay to keep this on the backlog. I think further design iteration, taking bandwidth consumption into consideration, needs to take place before we can scope the engineering effort.

@dr0ptp4kt agreed. @Nirzar might prioritize this for design work as I know the app teams were very happy with the results of enlargening app images ;)

Jdlrobson updated the task description. (Show Details)May 18 2016, 12:20 AM

Jdlrobson merged a task: T135426: Gallery resolution on mobile.

Jdlrobson added subscribers: phuedx, Dominikmatus, Zppix.

Jdlrobson renamed this task from images are too small on tablets and large phones to images are too small.May 18 2016, 12:24 AM

Jdlrobson updated the task description. (Show Details)

This is article on czech wikipedia in app. Look at that resolution.

Screenshot (11-19AM, May 19, 2016).png (1×720 px, 593 KB)

@Dominikmatus What phone is that? I'm guessing it has a hiDpi display, and IIRC not long ago hi-res images using srcset were disabled in mobile web in favor of a faster experience.

If the resulting experience is as bad as you show there, it is important that we document it and push to revert the srcset removal.

I'd encourage you to create a new bug about the blurry images and that you attach phone model and screenshots and see where that gets us. This task is not about that problem.

Thanks!

If the resulting experience is as bad as you show there, it is important that we document it and push to revert the srcset removal.

yes that change did impact our apps. but improved the web? in any case, ideally if we can serve up things based on the client, that would be great.

as far as the experience goes, bigger images is a feature on apps. bigger images make the article look media rich. combination of textual content with large images is a qualitative improvement.

Quiddity unsubscribed.May 19 2016, 4:49 PM

@Jhernandez note the apps have a specific workaround for this bug. They read srcset and expand the image. This image has no larger resolution and this is a hack so breaks. I thus agree with @Nirzar.

Here's how it looks on web (which is what we're talking about in this bug)
https://cs.wikipedia.org/wiki/Kostel_V%C5%A1ech_svat%C3%BDch_(P%C5%99eska%C4%8De)

Screen Shot 2016-05-19 at 11.40.38 AM.png (442×390 px, 62 KB)

Hello, I reported it, but somebody merged it with this. And, I'm using Samsung Galaxy S5 mini.

Quiddity renamed this task from images are too small to Images are too small in MobileFrontend.Jan 30 2017, 8:43 PM

Quiddity removed subscribers: • Jaredzimmerman-WMF, • wikibugs-l-list.

Jdlrobson changed the status of subtask T69709: Change default thumbnail size to 300px for all WMF wikis from Open to Stalled.Apr 19 2017, 6:42 PM

Jdlrobson moved this task from Needs Prioritization to Design on the Web-Team-Backlog board.Jun 2 2017, 8:47 PM

Jdlrobson edited projects, added Web-Team-Backlog (Design); removed Web-Team-Backlog.

Jdlrobson closed this task as a duplicate of T69709: Change default thumbnail size to 300px for all WMF wikis.

Jdlrobson closed subtask T67160: Refresh thumbnail style in Vector as Invalid.Jun 29 2017, 8:40 PM

Jdlrobson closed subtask T101046: [EPIC] Use MCS as parser for main content in mobile web as Declined.Dec 19 2019, 4:23 PM

Restricted Application added a subscriber: RhinosF1. · View Herald TranscriptDec 19 2019, 4:23 PM

Aklapper changed the status of subtask T69709: Change default thumbnail size to 300px for all WMF wikis from Stalled to Open.May 19 2020, 10:42 AM

	F4031668: Screen Shot 2016-05-19 at 11.40.38 AM.png
	May 19 2016, 6:42 PM

	F4030168: Screenshot (11-19AM, May 19, 2016).png
	May 19 2016, 9:25 AM

	F4025495: Screen Shot 2016-05-17 at 5.20.11 PM.png
	May 18 2016, 12:20 AM

	F13333: Screenshot_2014-03-26-14-53-20.png
	Aug 4 2015, 6:57 PM

	F13339: Screenshot_2014-03-26-15-17-45.png
	Nov 22 2014, 3:05 AM

	F13338: Screenshot_2014-03-26-15-16-46.png
	Nov 22 2014, 3:05 AM

	F13336: Screenshot_2014-03-26-15-20-03.png
	Nov 22 2014, 3:05 AM

	F13337: Screenshot_2014-03-26-15-22-31.png
	Nov 22 2014, 3:05 AM

Images are too small in MobileFrontendClosed, DuplicatePublicActions

Description

Larger devices

Galleries

Details

Related ObjectsSearch...

Event Timeline

Images are too small in MobileFrontend
Closed, DuplicatePublic
Actions

Related Objects
Search...