Last modified: 2014-11-20 09:21:30 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T33366, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 31366 - File description page thumbnail click versus EXIF rotation, mustRender()
File description page thumbnail click versus EXIF rotation, mustRender()
Status: NEW
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
1.22.0
All All
: Normal normal with 3 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 31504
  Show dependency treegraph
 
Reported: 2011-10-05 03:31 UTC by Tim Starling
Modified: 2014-11-20 09:21 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Patch (2.65 KB, patch)
2011-10-07 20:19 UTC, Bryan Tong Minh
Details

Description Tim Starling 2011-10-05 03:31:41 UTC
Clicking the large 800px thumbnail on a file description page leads to the source image being viewed as if it were a page in the browser. This kind of made sense as a quick hack for enlarged views when image resize was first introduced. But for files with mustRender(), the behaviour is undesirable, since the browser's behaviour when the user clicks on that link is typically surprising and unpredictable.

This came up on IRC recently when a user complained that clicking on a description page thumbnail which has been rotated using the new EXIF autorotation feature resulted in an incorrectly rotated enlarged image being displayed. Such images have mustRender() == true.

I suggest making thumbnail clicks for files with mustRender() == true do nothing, i.e. don't wrap that image with a link. Instead, provide a text link to the source file in the area below the thumbnail. 

Ideally thumbnail clicks of images with mustRender() == false would provide some kind of sensible, interactive enlargement interface, rather than causing all navigation to disappear and instructing the browser to download and decompress potentially hundreds of megabytes of image data.
Comment 1 Brion Vibber 2011-10-05 17:36:45 UTC
This is the same way we already treat SVG, PDF, DjVu, TIFF, and other files that may or may not render in a browser... Might want to redo a lot of stuff on the image page. (I think there's some old bug entries along these lines to find...)
Comment 2 Brion Vibber 2011-10-05 19:21:21 UTC
bug 24619 -- requests a consistent full-size PNG download link for SVGs
bug 23162 -- requests the multiple size selector links for SVGs (some JS magic on some sites?) and other file types
bug 25148 -- requests making a dedicated 'download original file' link that can actually force a download
bug 29967 - requests a popup dialog to encapsulate the many sorts of options noted above
Comment 3 Bryan Tong Minh 2011-10-07 20:19:37 UTC
Created attachment 9186 [details]
Patch

Patch that removes the link on the thumbnail and adds a "full version" link to the image sizes. The full version link is the rotated image, unfortunately not yet losslessy rotated. The original files is downloadable via the file name. That one should probably be renamed to "original file" or something.
Comment 4 Saibo 2011-10-07 21:39:32 UTC
"removes the link on the thumbnail". No. The user is used to get the full version by clicking on the thumbnail. Do not change this for EXIF rotated images only. This is reduced functionality to "fix" a bug.

The full version link may be used by external tools - please think of this.
Comment 5 Saibo 2011-10-13 20:45:14 UTC
(In reply to comment #4)
> "removes the link on the thumbnail". No. The user is used to get the full
> version by clicking on the thumbnail. Do not change this for EXIF rotated
> images only. This is reduced functionality to "fix" a bug.
> 
> The full version link may be used by external tools - please think of this.

Ok, maybe this is okay for now. However, I do not know if some tools expect the original version link at the 800px image...
Comment 6 Sumana Harihareswara 2011-11-09 20:45:45 UTC
Adding "patch" and "reviewed" keywords for consistency, since this bug includes a reviewed patch.
Comment 7 Gerrit Notification Bot 2013-09-28 17:41:59 UTC
Change 86383 had a related patch set uploaded by Brian Wolff:
Make image on desc page not clickable if $file->mustRender() true.

https://gerrit.wikimedia.org/r/86383
Comment 8 Gerrit Notification Bot 2013-10-20 20:54:29 UTC
Change 86383 abandoned by Brian Wolff:
[DO NOT MERGE] Make image on desc page not clickable if $file->mustRender() true.

Reason:
Feedback on commons was negative

https://gerrit.wikimedia.org/r/86383
Comment 9 Tisza Gergő 2014-11-19 00:10:19 UTC
This could be solved by linking to a full-size thumbnail instead of the original file if mustRender() is true. See bug 65383 for a more generic discussion of showing thumbnails instead of originals.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links