Re: [Rhythmbox-devel] Rhythmbox pre-release, and landing of generic audio player write support

[Ed Catmur <ed catmur co uk>]

On Mon, 2007-02-12 at 22:07 +0000, Ed Catmur wrote:
> On Mon, 2007-02-12 at 14:35 -0500, William Jon McCann wrote:
> > On 2/12/07, Ed Catmur <ed catmur co uk> wrote:
> > > On Mon, 2007-02-12 at 11:24 -0500, Andrew Conkling wrote:
> > > > On 2/12/07, Ed Catmur <ed catmur co uk> wrote:
> > > > > On Mon, 2007-02-12 at 01:35 -0500, Andrew Conkling wrote:
> > > > > > It's a tasteful X, but I'd prefer nothing:
> > > > > > http://img361.imageshack.us/img361/975/rbcoverartzb8.png
> > > > >
> > > > > Ah.  That's wrong; it should be the Rhythmbox icon.  Currently it uses
> > > > > "gnome-media-player"; maybe that should be changed to "rhythmbox"?
> > > >
> > > > Not sure, but apparently (if only in Ubuntu) the icon isn't being
> > > > registered correctly.
> > > Hm - "gnome-media-player" isn't in hicolor, but "rhythmbox" is.  This
> > > might be the problem.
> > 
> > http://svn.gnome.org/viewcvs/gnome-icon-theme?rev=1112&view=rev
> > 
> > Looks like that icon has been axed.
> 
> Right, yeah.  This patch against current svn uses RB_APP_ICON from
> rb-stock-icons.[ch], which was always the correct thing to do - and uses
> it in notifications as well.

In bugzilla: http://bugzilla.gnome.org/show_bug.cgi?id=407246