After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 384483 - Open cover in Image Viewer
Open cover in Image Viewer
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Plugins (other)
0.9.6
Other Linux
: Normal enhancement
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 513187 (view as bug list)
Depends on: 363694 411560 411565
Blocks: 412853
 
 
Reported: 2006-12-10 19:46 UTC by Sven Arvidsson
Modified: 2010-01-24 08:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
33cover-image-viewer.patch (2.69 KB, patch)
2007-02-20 19:00 UTC, Ed Catmur
none Details | Review
33cover-image-viewer.patch (3.85 KB, patch)
2007-02-24 13:38 UTC, Ed Catmur
none Details | Review
33cover-image-viewer.patch (3.97 KB, patch)
2007-02-24 18:38 UTC, Ed Catmur
none Details | Review
33cover-image-viewer.patch (3.64 KB, patch)
2007-02-24 23:53 UTC, Ed Catmur
none Details | Review
33cover-image-viewer.patch (3.63 KB, patch)
2007-02-25 20:29 UTC, Ed Catmur
none Details | Review
33cover-image-viewer.patch (3.65 KB, patch)
2008-03-24 21:10 UTC, Ed Catmur
none Details | Review

Description Sven Arvidsson 2006-12-10 19:46:14 UTC
I find myself increasing the size of the side pane from time to time, to get a better view of the cover art. An option to open the cover in the default image viewer would be highly appreciated.

Maybe a right click menu on the cover would be a good idea?
Comment 1 Alex Lancaster 2007-01-21 13:04:20 UTC
Some work on adding right-click as part of a cover art manager is going on at bug #363694.
Comment 2 Ed Catmur 2007-02-20 19:00:33 UTC
Created attachment 82985 [details] [review]
33cover-image-viewer.patch

Suggested patch.
Comment 3 Ed Catmur 2007-02-24 13:38:50 UTC
Created attachment 83235 [details] [review]
33cover-image-viewer.patch

Kill the gnome-open crack.  Note problems re bug 411560; don't think they matter too much.
Comment 4 Ed Catmur 2007-02-24 13:41:02 UTC
Remaining issues: we can't do startup notification, which is a pity.  Would it make sense to add a launch_with_startup_notification to lib, probably cut-and-pasted from nautilus_launch_application?
Comment 5 Ed Catmur 2007-02-24 13:45:09 UTC
Startup notification filed as bug 411565.
Comment 6 Ed Catmur 2007-02-24 18:38:04 UTC
Created attachment 83267 [details] [review]
33cover-image-viewer.patch

Oohie.  Use gobject.spawn_async, the way it should be.
Comment 7 Ed Catmur 2007-02-24 23:53:53 UTC
Created attachment 83277 [details] [review]
33cover-image-viewer.patch

Update to current state of bug 363694
Comment 8 Alex Lancaster 2007-02-25 12:15:11 UTC
Works for me.  One issue is that it seems to load/index all the images in the ~/.gnome2/rhythmbox/covers directory, because you can see the image position in the list of Eye of GNOME e.g. "345/1200".  Is this intentional?
Comment 9 Ed Catmur 2007-02-25 17:13:36 UTC
Nah, that's just what EOG does; it thinks it's a directory browser as well as an image viewer.
Comment 10 Ed Catmur 2007-02-25 20:29:29 UTC
Created attachment 83325 [details] [review]
33cover-image-viewer.patch

Update again
Comment 11 Jonathan Matthew 2008-01-30 22:13:08 UTC
*** Bug 513187 has been marked as a duplicate of this bug. ***
Comment 12 Ed Catmur 2008-03-24 21:10:49 UTC
Created attachment 107948 [details] [review]
33cover-image-viewer.patch

Update to 0.11.5
Comment 13 Jonathan Matthew 2010-01-24 08:02:29 UTC
commit f27ba835d2cdac3679cd62c40732235a5cbee846
Author: Jonathan Matthew <jonathan@d14n.org>
Date:   Sun Jan 24 18:00:19 2010 +1000

    artdisplay: open the image in the default image viewer on doubleclick (bug #384483)
    
    This isn't particularly discoverable, but I'm not sure it needs to be.