Page MenuHomePhabricator

Offer a way to disable MediaViewer for certain images
Closed, ResolvedPublic

Description

Migrated from: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/511

Narrative

As an editor, I can disable Media Viewer for images that are not suitable for this tool, so that users are not confused.

A number of image files need to be excluded, because they are not suitable for Media Viewer (such as maps using weird CSS/JS tricks, or images which use a clipping template). Many community members have reported this issue, which delivers an unpleasant browsing experience for users who only want to view images that are relevant and render well in Media Viewer.

Acceptance Criteria

  • Enable community members to use a special class name ('noviewer') to exclude an image from Media Viewer, as shown in this example: [ [ File:Foo.png ] ] </span>
  • This CSS class would disable Media Viewer in the same way that we already exclude images which have a 'metadata' class
  • Should also work with[ [ File:Foo.png | class="noviewer" ] ]

Notes

The reason this is needed is that there are images that should not be excluded via the 'metadata' class but still unsuitable for Media Viewer

To find an acceptable class name, we started this onwiki discussion on MediaWiki.org.

The 'noviewer' label seemed most popular with participants, many of whom also thought this feature is requiredto filter out unsuitable images.

See also:

Related Bugs

Related Stories

#8 Media Viewer Front-end

Related Changesets

Event Timeline

MingleTerminator raised the priority of this task from to High.
In mingle on 2014-06-09 at 19:23:09, @Tgr wrote:

Turns out there is a class parameter for images, so there is a nicer syntax:

[[File:Foo.png|class=noviewer]]

This puts the class directly on the img element, we should make sure we support that.

In mingle on 2014-06-09 at 19:23:38, @Tgr wrote:

Meh.

[ [ Foo.png | class=noviewer ] ]

In mingle on 2014-06-12 at 16:27:37, @MarkTraceur wrote:

Is this awaiting code review? There's no patch linked.