See http://en.wikipedia.org/wiki/Comparison_of_web_browsers#Image_format_sup...
the attached patch won't hurt.
not sure it'll applies but the modification is obvious
Ideally $file->type == 'image' should imply rendering, then it's up to the browser to handle the <img>
according to the mime type it supports.

CommentFileSizeAuthor
display-bmp-svg.patch809 bytesdrzraf

Comments

drzraf’s picture

drzraf commented
Title: add bmp and svg extensions to the file browser can render » add bmp and svg extensions to the mime type to be displayed within <img>
fasdalf@fasdalf.ru’s picture

fasdalf@fasdalf.ru commented

BMP is too-damn-huge and SVG is poorly supported by browsers, so i think it's bad idea.

drzraf’s picture

drzraf commented

I don't think there is room for arbitrary exclusion here.
If the browser does not support SVG it WILL offer to download the file or won't display it.
But not offering support for SVG by default is not a good choice, at least because some (very common) browsers DOES support (a large subset) of SVG.
BMP is actually an image format, please see grep image/ /etc/mime.types

fasdalf@fasdalf.ru’s picture

fasdalf@fasdalf.ru commented

I'm agree now. After all it worths applying a patch.

drzraf’s picture

drzraf commented

ping

drzraf’s picture

drzraf commented
Issue summary: View changes