GNOME Bugzilla – Bug 562072
Provide a screensaver from the user's cover art
Last modified: 2018-05-24 13:48:29 UTC
Something like collage, but with coverart: http://build.last.fm/item/455
Something like this: http://www.youtube.com/watch?v=srDkugKfGY0 Could probably be done using Clutter, using Fluttr as a base (needs porting to Clutter 0.8): http://git.clutter-project.org/cgit.cgi?url=toys/tree/fluttr
I'd like to implement the screensaver (using libclutter, indeed), but there are two architectural problems for me. * Cover arts support is implemented as a plugin, so the screensaver has to be a part of the plugin (doh!). * The artdisplay plugin is written in Python, so the cover search code sharing is problematic. In order to solve the second problem, I'm starting to port the plugin to C. The artdisplay is very popular, it is enabled by default, so, I guess, it is worth to make it run as a native code. And, this would allow to share the cover searching code for the screensaver. Does this make sense? Does this have a chance to be committed?
You can already request cover art for any database entry from another plugin. The ipod and mtp plugins do this. I don't really want to see the artdisplay plugin rewritten in C.
Created attachment 144005 [details] [review] Patch for this bug
Created attachment 158054 [details] [review] Add cover art screensaver
Review of attachment 158054 [details] [review]: A couple of comments: - the configure changes should probably have a --enable-screensaver=XXX to force the screensaver to be built (and exit if requested but clutter was not present) - the application itself uses an inordinate amount of memory, and took a long while to start up from cold (I would think up to 2 minutes), reading all 80GB of cover art I have - the cover art floating above the cover grid was in the screen's aspect ratio, instead of being square.
Created attachment 159114 [details] [review] Add cover art screensaver
Maxim, could you explain what you changed in the latest patch?
(In reply to comment #8) > Maxim, could you explain what you changed in the latest patch? I fix problems, that mentioned in review.
-- GitLab Migration Automatic Message -- This bug has been migrated to GNOME's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/rhythmbox/issues/666.