Page MenuHomePhabricator

MediaViewer confuses different instances of the same image on the same page
Open, Stalled, LowPublic

Description

Steps to reproduce:

  1. go to https://www.mediawiki.org/wiki/Multimedia/About_Media_Viewer
  2. Click on the first image of the How does Media Viewer work? section.

Observe that the caption is missing.

MediaViewer actually opened the first instance of the same image (somewhat higher on the page, has no caption), but this is not obvious and can be quite confusing when the caption has important information (e.g. legend for a chart is often placed there) and the user is expecting it.


Version: unspecified
Severity: normal

Details

Reference
bz62039

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:03 AM
bzimport added a project: MediaViewer.
bzimport set Reference to bz62039.
bzimport added a subscriber: Unknown Object (MLST).
Tgr created this task.Feb 28 2014, 8:22 AM

This was why we had MediaViewer/File:Foobar.jpg/6 instead of just MediaViewer/File:Foobar.jpg in the past - I made sure the index matched, so you got the same view no matter what.

Though we could at least be cognisant of the order when opening the lightbox. Seems not difficult.

Tgr added a comment.Feb 28 2014, 8:19 PM

That scheme had other problems (not stable enough, not to mention ugly :)
I would still go with MediaViewer/<filename>/<index relative to other thumbnails of the same file>, with MediaViewer/<filename>/1 being aliased to MediaViewer/<filename>.

Gilles triaged this task as Low priority.Nov 24 2014, 2:50 PM
Gilles added a subscriber: Gilles.
Jdforrester-WMF moved this task from Untriaged to Backlog on the Multimedia board.Sep 4 2015, 6:11 PM
Restricted Application added subscribers: Matanya, Aklapper. · View Herald TranscriptSep 4 2015, 6:11 PM

Mass-removing the Multimedia tag from MediaViewer tasks, as this is now being worked on by the Reading department, not Editing's Multimedia team.

Change 126153 had a related patch set uploaded (by Gergő Tisza):
[WIP] Add position handling to URLs

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

Jdlrobson changed the task status from Open to Stalled.Oct 9 2015, 12:59 AM
Jdlrobson added a subscriber: Jdlrobson.

Patch has been a WIP for some time now.

Tgr removed a subscriber: Tgr.Jul 9 2019, 6:05 PM