Page MenuHomePhabricator

MIDI files - inconsistent handling of [[File:...]] vs [[:File:...]] vs [[Media:...]]
Closed, ResolvedPublic

Description

Author: happy.melon.wiki

Description:
I'm not actually sure which component handles MIDI playback in MW, so I stuck this in the general file/repo, please move it if you know where it belongs.

There is currently inconsistent handling of links to media files between MIDI and OGG:

[[:File:Foo.ogg]] --> link to file description page, as expected
[[:File.Foo.midi]] --> link to file description page, as expected

[[Media:Foo.ogg]] --> Downloads file on FF3.0.10
[[Media:Foo.midi]] --> *plays* file in distinct window on FF3.0.10

[[File:Foo.ogg]] --> Inline widget to play file inline
[[File:Foo.midi]] --> links to file description page

Without a doubt, [[File:Foo.midi]] should be either an inline widget or a link to the same play-in-separate-window process that's currently achieved with [[Media:Foo.midi]]. I realise that the variation between the [[Media:...]] calls may be browser-dependent, but that's also something that should be looked into. These functions should have consistent handling.


Version: 1.15.x
Severity: minor

Details

Reference
bz18922

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:38 PM
bzimport set Reference to bz18922.

[[Media:Foo.ogg]] --> Downloads file on FF3.0.10
[[Media:Foo.midi]] --> *plays* file in distinct window on FF3.0.10

This is because your Firefox 3.0.10 installation doesn't have a native or plugin component installed to play .ogg files, but does have a plugin component to play .midi files. If your Firefox installation had a plugin capable of playing Ogg files, or you had Firefox 3.5 beta installed which supports them natively, they would happily play it inline the same way.

[[File:Foo.ogg]] --> Inline widget to play file inline
[[File:Foo.midi]] --> links to file description page

This is because the wiki you're looking at has an inline Ogg player component, but not an inline Midi player component... I'm duping this to bug 18852 which appears to be asking for just that.

  • This bug has been marked as a duplicate of bug 18852 ***