Image blobs should be printed as images instead of hexdumps

Ferry Huberts mailings at hupie.com
Thu Mar 1 14:34:13 CET 2012



On 01-03-12 10:56, Lauri Niskanen wrote:
> On 29/02/12 13:39, Ferry Huberts wrote:
>>
>>
>> On 29-02-12 12:30, Lauri Niskanen wrote:
>>> Hi!
>>>
>>> Cgit currently prints a hexdump when you look at a image blob (as with
>>> any other binary blob). This might be a good way to show generic binary
>>> files, but I think images (at least png, jpg) should be shown as images.
>>>
>>> Example of the current behaviour:
>>> http://hjemli.net/git/cgit/tree/cgit.png
>>>
>>> I suggest that instead of the hexdump you would have HTML like this:
>>> <img src="http://hjemli.net/git/cgit/plain/cgit.png" />
>>>
>>> SVG images are not binaric so I would leave them printed as text.
>>>
>>> What do you think about this?
>>>
>>
>>
>> This is kind-of solved by my mime patch that's on wip.
>> Once Lars merges that into stable, you'll have it
>>
>> http://hjemli.net/git/cgit/commit/?h=wip&id=d01c600c179593a53162a9d4e3040ecfc5078fdc
>>
>>
>>
>>
>> From what I understood from Lars is that files should always be shown
>> within cgit (non-plain) first, because some sites want to disable the
>> viewing of plain files.
>
> Thanks, but how does that patch solve my problem? Yeah, we probably need
> the mimetype information so that we know which files are images, but I
> don't see any code that would show images as <img>-tags instead of a
> hexdump.
>


from the page you mention, you can click the 'plain' link (if it's enabled).

That will send the file to you with the most appropriate mime-type.

-- 
Ferry Huberts




More information about the CGit mailing list