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 attached. 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 works: http://www.matroska.org/downloads/shellextension/
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: http://rapidshare.de/files/35094692/howlsmovingcastle_preview.mkv.html (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) and GST_TAG_IMAGE - 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: http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gst-plugins-base-libs/html/gst-plugins-base-libs-gsttag.html#GstTagImageType 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 <hadess@hadess.net> * configure.in: Require gstreamer-tag as well * src/backend/bacon-video-widget.h: * src/backend/bacon-video-widget-gst-0.10.c (bacon_video_widget_get_metadata_pixbuf), (bacon_video_widget_get_best_image), (bacon_video_widget_get_metadata): Add code to read covers from video and audio files, through the tags * src/backend/bacon-video-widget-xine.c (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 (Closes: #318748)