GNOME Bugzilla – Bug 318748
Display attached cover instead of thumbnail (when possible)
Last modified: 2008-06-10 21:19:08 UTC
Some (newer) container formats allows to ad attachements or still images into
the file, like: matroska, mpeg4, ratdvd (and maybe others too).
In the most of such files there are (can be) covers from the movie or music
Totem-thumbnailer should be display the attached cover (mostly: cover.jpg)
instead of a thumbnail, when an attached cover is avaible.
For matroska files, there is a Shell Extension for Windows avaible, that did the
same (display cover instead of thumbnail if avaible). Here is explained, how it
I've implemented this for quicktime elsewhere, although the thumbnailer needs some minor work to use this. Adding it to matroska is a matter of doing it. Same is true for id3.
xine-lib doesn't support that yet, but I'd be happy to support it for some specific file types.
Do you have any MPEG4 videos with such thumbnails?
reviczky: can you please attach a testcase as requested by bastien?
(In reply to comment #3)
> reviczky: can you please attach a testcase as requested by bastien?
Here is a sample Matroska file wich an attached cover inside:
(If there is a place, where i can upload test files, than let me know.)
*** Bug 339703 has been marked as a duplicate of this bug. ***
Probably it's a relelated bug, but matroska(and mp4) embedded chapters aren't recornized.
Here is the specification http://www.matroska.org/technical/specs/chapters/index.html
For chapter support see bug #163546: http://bugzilla.gnome.org/show_bug.cgi?id=163546
Created attachment 112498 [details] [review]
Support thumbnailing audio files
Quick one. We also need to exit earlier when there's no cover, and the file is audio only.
Quick comment as requested: GStreamer bits look ok, but not perfect:
- there's both GST_TAG_PREVIEW_IMAGE (which is supposed to be icon sized)
- there may be only one GST_TAG_PREVIEW_IMAGE per taglist, but there
can be multiple GST_TAG_IMAGEs
- GST_TAG_IMAGEs may/should have an "image-type" field in the buffer caps,
which is an enum:
There's probably a utility function for libgsttag in there somewhere ... :)
Doesn't work with Matroska files (see bug 537622), and doesn't work with the xine-lib backend (because it doesn't support exporting that data).
Thanks Tim for the help.
2008-06-10 Bastien Nocera <firstname.lastname@example.org>
* configure.in: Require gstreamer-tag as well
(bacon_video_widget_get_metadata): Add code to read covers from
video and audio files, through the tags
(bacon_video_widget_get_metadata): Implement stubs of the above
* src/totem-video-thumbnailer.c (capture_interesting_frame),
(capture_frame_at_time), (has_audio), (on_got_metadata_event),
(main): Implement saving the cover of a video/audio file