Closed chrysn closed 4 years ago
Adding .mkv and .mka (Matroska) files.
Eventually, it'd be nice to use an externally maintained database, and ideally use media types (<getcontenttype>
), but those are ones I found on my server.
Yeah, agreed. In the latest changes I removed the mimeType checking as that was conflicting with the approach I wanted to take for allowing the previews to work, but it might be nice to default back to using those. Before changing that too much though, I think I'll replace the lightbox and tackle type checking as part of that process.
This looks good to go though! I didn't realise that WebMs are in a Matroska container too... Learned something thanks to this PR! :)
These files can be played (at least by Firefox on Linux, didn't test others), and are widely used (eg. for high quality recordings).