Page MenuHomePhabricator

Need error message and/or better design for thumbnails broken due to megapixel limit
Closed, ResolvedPublic

Description

Author: mcdevitd

Description:
https://commons.wikimedia.org/wiki/File:Constitution_of_the_United_States,_page_1.tif is a high-resolution TIFF file which is above the current megapixel limit for rendering thumbnails.

As a result, image description pages just render a large broken thumbnail image, but do not communicate to the user what the problem is. In particular, since this is how the image will always render due to its large resolution, not a temporary issue, the design could be improved so it is not so confusing to the user (and so it is clearer that they can download the original file if desired).


Version: unspecified
Severity: normal

Details

Reference
bz62306

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

bzimport raised the priority of this task from to Normal.
bzimport set Reference to bz62306.
bzimport added a subscriber: Unknown Object (MLST).
bzimport created this task.Mar 6 2014, 6:33 AM

PagedTiffHandler checks size limits at wrong stage (not that mw default behaviour is any better)

Change 135101 had a related patch set uploaded by Brian Wolff:
Check $wgMaxImageArea before downloading file asset from swift

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

(In reply to Gerrit Notification Bot from comment #2)

Change 135101 had a related patch set uploaded by Brian Wolff:
Check $wgMaxImageArea before downloading file asset from swift

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

This will make the error message be the same as other files get (e.g. GIFs) namely: https://commons.wikimedia.org/wiki/MediaWiki:Thumbnail_invalid_params

I think we need a less hacky system for dealing with megapixel limit. See bug 32387 for that issue.

Change 135101 merged by jenkins-bot:
Check $wgMaxImageArea before downloading file asset from swift

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

Calling this fixed as there now actually is an error (look for it on commons on tuesday). For general better error handling of this situation see bug 32387

mcdevitd wrote:

(In reply to Bawolff (Brian Wolff) from comment #5)

Calling this fixed as there now actually is an error (look for it on commons
on tuesday). For general better error handling of this situation see bug
32387

Would this fix bug 65552 as well?

Gilles moved this task from Untriaged to Done on the Multimedia board.
Gilles raised the priority of this task from Normal to Unbreak Now!.
Gilles lowered the priority of this task from Unbreak Now! to Normal.Dec 4 2014, 11:21 AM