Page MenuHomePhabricator

Show error message when MMV is opened via an URL hash that points to a non-existing image
Closed, ResolvedPublic

Description

Currently the user just gets a black screen.

When the image cannot be loaded, inform the user the image doesn't exist.

Also interesting:

  • Image was renamed.
  • Image still exists but removed from article (currently ignored, we just show the image directly).

Mingle:

Event Timeline

MingleTerminator raised the priority of this task from to Low.Dec 8 2014, 5:01 PM
MingleTerminator added a project: Multimedia.

The black screen is cleared up now; an error message would still be useful. (For extra points, point the user to the file page if it exists. See also T77308.)

Krinkle set Security to None.
Krinkle subscribed.
In T77327#840306, @Tgr wrote:

The black screen is cleared up now; an error message would still be useful. (For extra points, point the user to the file page if it exists. See also T77308.)

The black screen is still created on-load and then removed half a second later. This flash can be confusing for users (not sure what it does in terms of accessibility, either).

Not sure what would be a better alternative; if we don't show anything the user thinks it is not working and clicks on other things and we end up with a mess.

FWIW there is an error message now via mw.notify which does not look great but at least gives an idea of what's happening.

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

Jdlrobson raised the priority of this task from Low to Medium.May 9 2023, 6:59 PM
Jdlrobson subscribed.

Change 921616 had a related patch set uploaded (by Simon04; author: Simon04):

[mediawiki/extensions/MultimediaViewer@master] Notify user and close MMV if image from route not present on page

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

Change 921616 merged by jenkins-bot:

[mediawiki/extensions/MultimediaViewer@master] Notify user and close MMV if image from route not present on page

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

matmarex assigned this task to simon04.