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 380896 - Tango icons for Rhythmbox and other Players
Tango icons for Rhythmbox and other Players
Status: RESOLVED OBSOLETE
Product: rhythmbox
Classification: Other
Component: User Interface
HEAD
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 426629 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-11-30 13:53 UTC by Michael Monreal
Modified: 2018-05-24 12:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
feed-podcast in 16x16 and 22x22 (5.34 KB, application/x-gzip)
2007-01-07 12:27 UTC, Andreas Nilsson
  Details
playlist in 16x16 and 22x22 (5.27 KB, application/x-gzip)
2007-01-07 14:39 UTC, Andreas Nilsson
  Details
playlist in 16x16, 22x22, 32x32 and 48x48 (7.70 KB, application/x-gzip)
2007-01-07 16:49 UTC, Andreas Nilsson
  Details
playlist-automatic in 16x16 and 22x22 (5.04 KB, application/x-gzip)
2007-01-07 17:21 UTC, Andreas Nilsson
  Details
radio-station and radio-station-new in 16x16 and 22x22 (5.39 KB, application/x-gzip)
2007-01-07 23:43 UTC, Andreas Nilsson
  Details
internet-radio and internet-radio-new (46.16 KB, application/x-gzip)
2007-01-10 15:45 UTC, Andreas Nilsson
  Details
use internet-radio and internet-radio-new icons (4.76 KB, patch)
2007-03-11 21:22 UTC, Jonathan Matthew
committed Details | Review
use new library-podcast and playlist* icons (5.59 KB, patch)
2007-03-16 15:33 UTC, Luca Ferretti
none Details | Review
A rhythmbox/trunk screenshot using new icons (132.46 KB, image/png)
2007-03-16 15:34 UTC, Luca Ferretti
  Details
updated patch (11.08 KB, patch)
2007-03-16 18:36 UTC, William Jon McCann
none Details | Review
icons taken from gtk (4.73 KB, application/x-bzip)
2007-04-05 17:38 UTC, David Prieto
  Details
Application icon (13.99 KB, application/x-gzip)
2007-04-06 15:16 UTC, Andreas Nilsson
  Details
without the blue waves for "not playing" (10.77 KB, application/x-gzip)
2007-04-26 08:59 UTC, Andreas Nilsson
  Details
use more of these icons (27.60 KB, patch)
2007-05-05 03:10 UTC, Jonathan Matthew
committed Details | Review
new and moved icons (17.64 KB, application/x-compressed-tar)
2007-05-05 03:11 UTC, Jonathan Matthew
  Details
playlist automatic in 16x16, 22x22, 32x32 and 48x48 (7.89 KB, application/x-gzip)
2007-05-10 10:15 UTC, Andreas Nilsson
  Details
Various *-new icons (38.30 KB, application/x-gzip)
2007-05-30 12:10 UTC, Michael Monreal
  Details
patch to use them (5.52 KB, patch)
2007-05-30 12:49 UTC, Jonathan Matthew
committed Details | Review
tar file containing the icons (38.73 KB, application/x-compressed-tar)
2007-05-30 12:52 UTC, Jonathan Matthew
  Details
22x22 icons with diferent star emblem (9.67 KB, application/x-gzip)
2007-05-30 19:19 UTC, Michael Monreal
  Details
Last.fm, Jamendo and Magnatune icons (10.09 KB, application/x-gzip)
2007-06-02 19:10 UTC, Michael Monreal
  Details
Last.fm, Jamendo and Magnatune icons (v2) (9.92 KB, application/x-bzip)
2007-06-02 23:10 UTC, Alexander “weej” Jones
  Details
Screenshot of new Last.fm, Jamendo and Magnatune icons in use (229.78 KB, image/png)
2007-06-02 23:22 UTC, Alexander “weej” Jones
  Details
HighContrast icons (3.04 KB, application/x-gzip)
2007-06-03 00:06 UTC, Andreas Nilsson
  Details
HighConstast Rhythmbox (127.37 KB, image/png)
2007-06-04 10:56 UTC, Luca Ferretti
  Details
1st try of a new library icon (4.57 KB, application/x-gzip)
2007-06-07 10:26 UTC, Michael Monreal
  Details
make magnatune and jamendo icons themeable (6.24 KB, patch)
2007-06-23 06:21 UTC, Jonathan Matthew
committed Details | Review

Description Michael Monreal 2006-11-30 13:53:33 UTC
Rhythmbox currently uses a few outdated icons that should be replaced by Tango icons. It would be nice to take the opportunity and create a "media players" set similar to ArtLibreSet¹, with icons that can be used by other players, too.

Icons that are currently needed by Rhythmbox:

Application icon (all sizes)
=> needs a nice metaphor

Tray icon (22x22)
=> should perhaps reuse the application icon?
=> should perhaps provide a disabled state for "not playing"?

Toolbar (24x24)
- Show/Hide Browser (I vote for removing this toggle button!)

Sidebar (currently 24x24, but should better be 16x16 to match Nautilus sidebar)
- Library (Pile/Shelf of books or CDs?)
- (Static) Playlist (Something reflecting a list of songs)
- Automatic Playlist (Like Playlist, but also reflect "automatic")
- Song queue (action reflecting enqueing or playing after another? hard...)
- Podcast (Look what Banshee² and iTunes³ have, should probably be similar)
- Internet radio
- Last.fm radio (Can the Last.fm logo be used? Something not too different?)
- DAAP share (Perhaps reuse the BNC connector from the current shared folders and something reflecting music on top of that?)
- Eject (To eject CDs and DAPs -> look at your stereo)


[1] http://tango.freedesktop.org/ArtLibreSet
[2] http://jimmac.musichall.cz/wipicons/banshee//podcast.png
[3] http://images.apple.com/itunes/store/images/podcast_hero_win20060912.jpg
Comment 1 James "Doc" Livingston 2006-12-01 14:29:37 UTC
(In reply to comment #0)
> Tray icon (22x22)
> => should perhaps reuse the application icon?

This really depends on what the application icon is, but in general I think it would be a good idea.

> => should perhaps provide a disabled state for "not playing"?

Whether this would work obviously depends on what the tray icon looks like.


> Sidebar (currently 24x24, but should better be 16x16 to match Nautilus sidebar)

I'll take a look at what it looks like with smaller icons.


> - Library (Pile/Shelf of books or CDs?)

There's some discussion about changing the source list so that "Library" is just a group header, with the Music source inside it. So we may not need this in the future.


> - Song queue (action reflecting enqueing or playing after another? hard...)

I'm not sure what the icon would look like, but one for this would be nice.


> - Last.fm radio (Can the Last.fm logo be used? Something not too different?)

It should probably use a small version of the logo - I've been meaning to look at it, but hadn't found the time.


> - Eject (To eject CDs and DAPs -> look at your stereo)

Tango already has one of these, and we use it.
Comment 2 Andreas Nilsson 2007-01-07 12:27:40 UTC
Created attachment 79618 [details]
feed-podcast in 16x16 and 22x22

I was thinking of using those from Banshee first, but those are MIT-licensed as Banshee itself, so I drew a gpl one. (or is it ok for rmbox to ship mit-icons?)
Comment 3 Andreas Nilsson 2007-01-07 14:39:57 UTC
Created attachment 79634 [details]
playlist in 16x16 and 22x22
Comment 4 Andreas Nilsson 2007-01-07 16:49:26 UTC
Created attachment 79645 [details]
playlist in 16x16, 22x22, 32x32 and 48x48
Comment 5 Andreas Nilsson 2007-01-07 17:21:07 UTC
Created attachment 79651 [details]
playlist-automatic in 16x16 and 22x22
Comment 6 Andreas Nilsson 2007-01-07 23:43:22 UTC
Created attachment 79695 [details]
radio-station and radio-station-new in 16x16 and 22x22

Sorry for the flood. :)
Comment 7 Andreas Nilsson 2007-01-10 15:45:41 UTC
Created attachment 79957 [details]
internet-radio and internet-radio-new

In all sizes made by Josef Vybíral.
Comment 8 Luca Ferretti 2007-01-24 15:46:56 UTC
@Michael and James

A MediaLibreSet could be a good idea, someone should join tango mailin list and propose something. But IMHO this icon set should be more generic, more multimedia then music. This mean include stuff related to video, audio and picture (both playing and production) and keep specific stuff in RB (for example ban/love in last.fm). A good example of generic multimedia icon set come from stockicons.com - see for example http://stockicons.com/detail/193

By now Rhythmbox could use the custom themeable app icons trick: see http://live.gnome.org/ThemableAppIcons for more info. Using the proper names for custom named icons, we will be able to switch to future MediaLibreSet when it will be ready.

By now IMHO rhythmbox can provide those icons:
      * apps/rhythmbox
      * places/playlist
      * places/playlist-automatic
      * places/library-music
      * places/library-internet-radio
      * places/library-podcast
      * places/music-share
      * places/play-queue
      * actions/playlist-new
      * actions/playlist-automatic-new
      * actions/internet-radio-new
      * actions/podcast-feed-new
      * actions/import-file
      * actions/import-folder
      * actions/love-song (last.fm)
      * actions/ban-song  (last.fm)
      * actions/skip-song (last.fm)
      * status/browser-visible

This is just a starting list, using contexts and name scheme from Icon Naming Spec.

@James
I checked the current stock icons code in rhythmbox sources. It's really outdated and don't use any named icon facility. I think we need to rewrote it from scratch. Of course we have to define in advance which stock items Rhythmbox will need. I think using something like in current Epiphany code is the best option.

@Andreas
Not sure, but did you added a gradient for each "row" ub playlist icons? If so, isn't better use flat colors for each row and put a single light (white to transparent, top to bottom) gradient over all rows?
Comment 9 Andreas Nilsson 2007-01-29 12:17:46 UTC
@Luca
Nope, they are just semi-transparent objects on top of a gradient, so they pick up the "light" from the object beneath.
Comment 10 James "Doc" Livingston 2007-01-29 12:30:22 UTC
Sorry for taking so long to comment on this. The icons look pretty cool.

Rewriting out icon-handling to do things the proper way would be good idea. It would also make it possible for plugins to add an "icons" dir under their global and user-specific directories for plugin-specific art.


Regarding the icons from Banshee, using MIT-licenced stuff is fine, although I don't think it actually matters too much for most licences (that deal with code issues) since we don't actually "link" the art to the program.
Comment 11 Jonathan Matthew 2007-03-11 21:15:11 UTC
As part of some other work, I've added two directories to the gtk icon search path - $pkgdatadir/icons (/usr/share/rhythmbox/icons, typically) and ~/.gnome2/rhythmbox/icons/, which allows plugins to install their own icons.
Comment 12 Jonathan Matthew 2007-03-11 21:22:42 UTC
Created attachment 84400 [details] [review]
use internet-radio and internet-radio-new icons

extract internet-radio.tar.gz (from comment 7) in plugins/iradio/icons/hicolor to get the actual icons in the right place.
Comment 13 James "Doc" Livingston 2007-03-15 10:42:47 UTC
Looks good to me, aside from a typo in the name of "internet-radio-new.png" (it's "intenet-radio-new.png"). I'd be happy for this and the other icons to go in, and get some feedback on the actual art on the mailing list.
Comment 14 Luca Ferretti 2007-03-16 13:27:05 UTC
(In reply to comment #13)
> Looks good to me, aside from a typo in the name of "internet-radio-new.png"
> (it's "intenet-radio-new.png"). I'd be happy for this and the other icons to go
> in, and get some feedback on the actual art on the mailing list.
> 

OK. I'll start committing them on trunk under data/icons and installing under $prefix/share/rhythmbox/icons/hicolor. I'll follow naming sheme in comment 8.

Comment 15 Luca Ferretti 2007-03-16 14:58:48 UTC
All icons in previous comments are now on trunk. I've also committed changes to plugins/iradio/rb-iradio-source.c from Johnathan (comment 12).

Missing sizes:
  scalable:  library-podcast, playlist-automatic
  32x32: library-podcast, playlist-automatic
  24x24: I'll create them from 22x22 later

@Andreas: just 0.2 - what about increasing the rows in large playlist icons? I mean, increase from 4 to 6 rows in 32x32 and from 2 to 8 in scalable. Larger icon, more entries :-)

@James: I didn't add credits for Andreas and Josef in ChangeLog. May I add the "artists"[1] property to GtkAboutDialog?

Forgot to mention: I think is better to place all common icons in a single hierarchy, instead splitting them in plugins/ directories, at least for always-build plugins as iradio. For example, the icon for visualizer plugin is inside plugins/visualizer directory: it's the right place, 'cause this plugin needs GST-10 to build. Same for future audioscrobbler icons: you can disable this plugin at configure time, so icons should be installed only if this plugin is enabled.

[1] http://developer.gnome.org/doc/API/2.0/gtk/GtkAboutDialog.html#GtkAboutDialog--artists
Comment 16 Luca Ferretti 2007-03-16 15:33:49 UTC
Created attachment 84720 [details] [review]
use new library-podcast and playlist* icons

I simply changed references to existing macros/defines/other. The code to lookup for icons in rb-stock-icons.c is still the same, so I'm not sure, for example, that 16x16 pixmaps will be used in Music->Playlists menu. It's more a workaround than a real solution to icon lookup.
Comment 17 Luca Ferretti 2007-03-16 15:34:59 UTC
Created attachment 84721 [details]
A rhythmbox/trunk screenshot using new icons
Comment 18 William Jon McCann 2007-03-16 18:36:06 UTC
Created attachment 84735 [details] [review]
updated patch

Updated this patch to have one place that we define the icon size.  So, none of the icons show up for me because we use 24pix icons for most everything.
Comment 19 William Jon McCann 2007-03-16 18:50:51 UTC
Oh, so apparently this 22pix thing was actually a design decision for the Tango themes:
http://tango.freedesktop.org/Generic_Icon_Theme_Guidelines#Small

48, 16, and umm 22.  Genius.
Comment 20 Andreas Nilsson 2007-03-17 00:12:54 UTC
William: the 22x22 size was decided because of compability with kde. We can therefore just add a 1 pixel around all images with imagemagick so it works well under gnome, clipping parts out of images on the other hand don't work. As luca said, he's going to generate those 24x24 images, unless I beat him to it.
Comment 21 William Jon McCann 2007-03-17 05:04:56 UTC
Andreas, I don't mean to offend anyone - the icons looks awesome.  But the logic for using 22x22 icons is really lacking.  You should have make you spec self consistent with multiples of 8.  As you point out KDE would have had an easy time adding a pixel to make their icons meet the spec.  However, GNOME has to recreate all of our icons because cropping isn't an option.  And in the process we make a really important GTK+ API useless.  Not a good design decision.  But whatever pointless now I suppose...
Comment 22 Alex Lancaster 2007-03-20 09:11:31 UTC
Since this patch was applied to SVN, the radio icon has completely disappeared from the source list (it can still be selected, but there's just no icon there).
Comment 23 James "Doc" Livingston 2007-03-21 07:55:04 UTC
The patch hasn't been committed yet. With the patch, some icons don't appear to work uninstalled (but work fine installed).
Comment 24 Alex Lancaster 2007-03-21 08:02:51 UTC
(In reply to comment #23)
> The patch hasn't been committed yet. With the patch, some icons don't appear to
> work uninstalled (but work fine installed).

Hmm, well, according to the ChangeLog, *something* was committed, because after this change:

2007-03-16  Luca Ferretti  <elle.uca@libero.it>

        * plugins/iradio/rb-iradio-source.c: (rb_iradio_source_init):
        Use new custom named icon for IRadio in source list and in
        NewRadio item. Original patch from Jonathan Matthew.

2007-03-16  Luca Ferretti  <elle.uca@libero.it>

        * configure.ac:
        * data/Makefile.am:
        * data/icons/*:
        Added new icons directory to provide custom themeable icons for
        Rhythmbox; see bug #380896 for detail. Also added available
        pixmaps from the same bug, but still unused in code.
 

my iradio icon disappeared even when running installed.
Comment 25 Alexander “weej” Jones 2007-03-27 18:25:18 UTC
Why aren't we using 16px source list icons? 22px is supposed to be for toolbars, I think.
Comment 26 David Prieto 2007-04-05 17:37:16 UTC
Is there any chance to see this icon, based on the new gtk-cdrom icon, used for the browser button?

http://img457.imageshack.us/img457/2577/screenshotcoldplaythescxz1.png
Comment 27 David Prieto 2007-04-05 17:38:39 UTC
Created attachment 85862 [details]
icons taken from gtk
Comment 28 Jonathan Matthew 2007-04-06 01:07:57 UTC
*** Bug 426629 has been marked as a duplicate of this bug. ***
Comment 29 Andreas Nilsson 2007-04-06 15:16:41 UTC
Created attachment 85909 [details]
Application icon

Application icon in all sizes by Hylke Bons and myself. Heavily based on Garretts icon from Bluecurve. Thought it was about time to finish it as it started popping up here and there on the net.
Comment 30 Jonathan Matthew 2007-04-07 13:16:33 UTC
We could use that for the tray icon too.  If we do that, we could also use a version without the blue sound wavy things to indicate playing/not playing status (bug 314608).
Comment 31 Andreas Nilsson 2007-04-26 08:59:11 UTC
Created attachment 87053 [details]
without the blue waves for "not playing"

Hope this works.
Comment 32 Jonathan Matthew 2007-05-04 14:25:42 UTC
Thanks, Andreas.

At 16x16, it's quite a subtle difference.  I barely notice it changing if I'm not looking directly at the icon and I already know what I'm looking for.  Maybe that's a good thing, though.  Once you know what you're looking for, a quick glance at the icon is all you need to figure out which state it's in, which is good.
Comment 33 Jonathan Matthew 2007-05-05 03:10:05 UTC
Created attachment 87584 [details] [review]
use more of these icons

- use the new app icon
- switch between 'rhythmbox' and 'rhythmbox-notplaying' for the tray icon depending on state (bug 314608)
- use the new podcast icon
- use the playlist and playlist-automatic icons
- move the rating star and eject icons into data/icons/
- only inline the star icons since we can live without the rest
- when --enable-uninstalled-build is used, add the source directory to the icon search path (need to do 'ln -s . hicolor' in data/icons/ for this to actually work for now)

tar file containing all the new and moved icons to follow.
Comment 34 Jonathan Matthew 2007-05-05 03:11:40 UTC
Created attachment 87585 [details]
new and moved icons

extract this in the root of the rhythmbox source tree.
Comment 35 Jonathan Matthew 2007-05-05 03:13:37 UTC
Does playlist-automatic exist in 32x32 and 'scalable' sizes?  It doesn't seem to appear in menus without the scalable version.

(the patch to use the iradio icons was committed some time ago)
Comment 36 James "Doc" Livingston 2007-05-08 13:38:02 UTC
The patch/tarball looks great to me, although the icons don't appear to work when running uninstalled.
Comment 37 Michael Monreal 2007-05-08 18:41:39 UTC
I unpacked tha tarball as described and applied the latest patch. Builds fine, make install works fine, started rhythmbox and most icons are missing now (playlists, queue....). James, is this the same you are seeing when running uninstalled?
Comment 38 Paul Drain 2007-05-10 03:09:28 UTC
(In reply to comment #37)
> I unpacked tha tarball as described and applied the latest patch. Builds fine,
> make install works fine, started rhythmbox and most icons are missing now
> (playlists, queue....). James, is this the same you are seeing when running
> uninstalled?
> 

If you're not using --enable-uninstalled-build, did you update the GTK+ icon cache?

If I 'make install' with this patch manually, the icons don't appear until i've done it.
Comment 39 Michael Monreal 2007-05-10 09:26:42 UTC
Well I did 

gtk-update-icon-cache -f /opt/gnome2.20/share/icons/hicolor

now, but the icons still don't show up.
Comment 40 Andreas Nilsson 2007-05-10 09:59:28 UTC
Jonathan: sorry, seems I forgot about those two sizes for that icon. It's in the works.
Comment 41 Andreas Nilsson 2007-05-10 10:15:54 UTC
Created attachment 87945 [details]
playlist automatic in 16x16, 22x22, 32x32 and 48x48

There that should be all. Just poke me if something else is missing.
Comment 42 Jonathan Matthew 2007-05-12 04:53:23 UTC
I've committed these new icons and the code changes to use them.
Comment 43 Jonathan Matthew 2007-05-12 07:13:17 UTC
I've also moved all the icons so they work in uninstalled builds.
Comment 44 Michael Monreal 2007-05-12 09:22:15 UTC
It still doesn't work for me... No icons for:

- Queue
- Podcast
- Radio
- Playlists
- Auto Playlists

Buth in the menus and in the main interface...
Comment 45 Baptiste Mille-Mathias 2007-05-12 09:24:04 UTC
Hello,

just a note to say it misses a tango icon for "missing files" 
Comment 46 Alex Lancaster 2007-05-14 07:10:40 UTC
(In reply to comment #44)
> It still doesn't work for me... No icons for:
> 
> - Queue
> - Podcast
> - Radio
> - Playlists
> - Auto Playlists
> 
> Buth in the menus and in the main interface...

I have this problem too, debugging on IRC with Jonathan we have found what appears to be an issue with the version of hicolor-icon-theme, places and status (where those icons are found) were only added in hicolor-icon-theme-0.10 and later according to:

http://webcvs.freedesktop.org/icon-theme/default-icon-theme/ChangeLog?revision=1.17&view=markup

Fedora Core 6 (for example) only ships with hicolor-icon-theme-0.9.2-1, so we need a workaround for older systems.

Comment 47 William Jon McCann 2007-05-14 12:00:39 UTC
Alex, that sounds right to me.  I was using 0.9 and I wasn't seeing the RB icons.  I updated to 0.10 and they work now.

I was getting 0.9 from jhbuild.  So, I've updated the moduleset to provide 0.10.

http://svn.gnome.org/viewcvs/jhbuild/trunk/modulesets/freedesktop.modules?r1=1439&r2=1503
Comment 48 Luca Ferretti 2007-05-30 10:27:51 UTC
I'm back :-) Any pending task?

I'm going to prepare the patch to add Artwork tab in About -> Credits. Could someone provide me the list of contributors?

Also it seems to me that playlist*-new are missing, isn't it?
Comment 49 Jonathan Matthew 2007-05-30 10:37:25 UTC
playlist-new, playlist-automatic-new, podcast-new.

If someone was to replace the visualisation icon I made with something that actually looks good, I'd be thankful.
Comment 50 David Prieto 2007-05-30 10:49:54 UTC
Some new icons for random, repeat and browse would be cool, too.

Fantastic work, BTW. Rhythmbox now looks as it deserves.
Comment 51 Michael Monreal 2007-05-30 12:10:31 UTC
Created attachment 89052 [details]
Various *-new icons

Seeing that it was me who filed the bug in the first place... here are he "-new" variants for playlist, playlist-automatic and podcast.
Comment 52 Jonathan Matthew 2007-05-30 12:49:51 UTC
Created attachment 89053 [details] [review]
patch to use them
Comment 53 Jonathan Matthew 2007-05-30 12:52:01 UTC
Created attachment 89054 [details]
tar file containing the icons

extract this in the rb source tree root to get the new *-new icons in the right places with the right names
Comment 54 Michael Monreal 2007-05-30 13:02:03 UTC
Argh... I also did the patch to use the new icons, but forgot to upload the second attachment, sorry :(
Comment 55 Luca Ferretti 2007-05-30 18:11:57 UTC
Attachments 89054, 89053 are now commited; 89052 should be deprecated by 89054.
Comment 56 Björn Martensen 2007-05-30 18:38:19 UTC
the stars on podcast-new and radio-new icons have different sizes in
rhythmbox from svn 5132. could you make them more consistent?
Comment 57 Michael Monreal 2007-05-30 19:19:30 UTC
Created attachment 89065 [details]
22x22 icons with diferent star emblem

@Björn: only in the 22x22 version. Are those even used somewhere? Anyway, I actually took the star from document-new in gnome-icon-theme, so this is the same size as the rest of gnome uses. But, at least for podcast in 22x22, the star seems a big bit...

Here's a tar which has both the radio-new with the big star, as well as the playlist-new, playlist-automatic-new and podcast-new with the smaller one. See what you like better.
Comment 58 Björn Martensen 2007-05-30 19:30:54 UTC
they are used in the tool bar.
Comment 59 Andreas Nilsson 2007-05-31 10:53:22 UTC
Made a page in the gnome wiki describing the names, this makes it easier to track the names when we're doing highcontrast icons (just started on the radio).
http://live.gnome.org/ThemableAppSpecificIcons/Rhythmbox
Please help fill out the Description/Usage field.
Comment 60 Michael Monreal 2007-06-02 19:10:36 UTC
Created attachment 89248 [details]
Last.fm, Jamendo and Magnatune icons

I redid the Last.fm, Jamendo and Magnatune icons using Tango look (but using the original colors). I think they look nice in rhythmbox as the size and general look is closer to the rest of the icons.
Comment 61 Alexander “weej” Jones 2007-06-02 23:08:03 UTC
Good effort, Michael.

My only comments would be to firstly check that we have the relevant rights within the branding licenses to manipulate these logos.

Secondly, the outside highlighting is a bit off-standard. This looks a bit too stripey. The outer highlight should be 20% value (as in Hue-Saturation-Value) of the base colour and the inner highlight should be much more subtle.

I've made some changes to the Last.fm and the Jamendo logos. The magnatune one needs a bit more work to get right.
Comment 62 Alexander “weej” Jones 2007-06-02 23:10:14 UTC
Created attachment 89256 [details]
Last.fm, Jamendo and Magnatune icons (v2)

So the only problem here is that plugins can't support icon themes yet. Tango-ifying these for now can't do much harm.

Also, these may need adjustments to actually fit 22x22. Currently, they are the old GTK size, 24x24.
Comment 63 Alexander “weej” Jones 2007-06-02 23:22:47 UTC
Created attachment 89259 [details]
Screenshot of new Last.fm, Jamendo and Magnatune icons in use
Comment 64 Jonathan Matthew 2007-06-02 23:29:45 UTC
Plugin-specific icons are themeable.  They just need to be placed in plugins/$plugin/icons/hicolor/$size/$context in the source tree and installed in $pkgdatadir/icons/hicolor/$size/$context like other icons.
Comment 65 Andreas Nilsson 2007-06-03 00:06:53 UTC
Created attachment 89262 [details]
HighContrast icons

HighContrast-icons for these actions. Perhaps I've should have opened a new bug about this, but lazy as I am I'm attaching them to this report instead ;)
Do we need a application icon as well?
Luca, I guess you're in charge of these.
Comment 66 Alexander “weej” Jones 2007-06-03 00:40:50 UTC
Jonathan

I meant that if the plugin is to be distributed standalone, for example, the icons can't be themeable (because the distribution is restricted to ~/.rhythmbox/plugins/$plugin/ or /usr/lib/rhythmbox/plugins/$plugin).

Maybe GTK has some support for "external" icon stores where a proper theme-dependent lookup procedure can take place. If it doesn't, it would be nice in the future.

Does someone want to start committing and updating the status of patches? There's a lot of work done here!
Comment 67 Jonathan Matthew 2007-06-03 00:50:46 UTC
Not that we really support separately distributed plugins all that much, but they could either install icons to ~/.gnome2/rhythmbox/icons or add their install location to the icon search path.
Comment 68 Michael Monreal 2007-06-03 08:12:49 UTC
Alex: I did those icons on 22x22 and then added 1px to each border because I was under the impression that 24x24 is actually needed here... for whatever reason, it looks right. Anyway, if 22x22 is needed that border can just be removed.

Concerning trademarked artwork, only the last.fm icon includes the original vektor "a" sign, if that is a problem it can easily be replaced. If general "likeness" is a problem, those icons could also be modified to just "similar" to the original artwork to be recognizable (what GNOME did with the acrobat logo after Adobe complained).
Comment 69 Luca Ferretti 2007-06-04 10:38:17 UTC
(In reply to comment #57)
> Created an attachment (id=89065) [edit]
> 22x22 icons with diferent star emblem
> 

Commited the new radio icon (bigger star) to match other new-* icons in gnome-icon-theme at similar size.
Comment 70 Luca Ferretti 2007-06-04 10:39:57 UTC
(In reply to comment #62)
> Created an attachment (id=89256) [edit]
> Last.fm, Jamendo and Magnatune icons (v2)
> 

Commited those icons replacing existing ones. Still not themeable (maybe I will work on it, it could be useful at least for a11y, not for "branding")
Comment 71 Luca Ferretti 2007-06-04 10:53:46 UTC
id=89262 id=89256 and id=89065 should be marked as commited
Comment 72 Luca Ferretti 2007-06-04 10:56:46 UTC
Created attachment 89322 [details]
HighConstast Rhythmbox

Awesome! 

PS please note the bad bad appearance of All/Artist/Album/Title selector in search bar :-( I'll open a bug.
Comment 73 Alex Lancaster 2007-06-04 11:36:18 UTC
Why did you remove the SVG version from SVN?:

$ svn up
D    data/icons/hicolor/22x22/actions/internet-radio-new.svg
U    data/icons/hicolor/22x22/actions/Makefile.am
A    data/icons/hicolor/22x22/actions/internet-radio-new.xcf
U    data/icons/hicolor/22x22/actions/internet-radio-new.png

This breaks the compilation:

Making all in actions
make[6]: Entering directory `/home/alex/build/rhythmbox/data/icons/hicolor/22x22/actions'
make[6]: *** No rule to make target `internet-radio-new.svg', needed by `all-am'.  Stop.
make[6]: Leaving directory `/home/alex/build/rhythmbox/data/icons/hicolor/22x22/actions'
make[5]: *** [all-recursive] Error 1
make[5]: Leaving directory `/home/alex/build/rhythmbox/data/icons/hicolor/22x22'
make[4]: *** [all-recursive] Error 1
make[4]: Leaving directory `/home/alex/build/rhythmbox/data/icons/hicolor'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/home/alex/build/rhythmbox/data/icons'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/alex/build/rhythmbox/data'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/alex/build/rhythmbox'
make: *** [all] Error 2
Comment 74 Luca Ferretti 2007-06-04 11:46:58 UTC
(In reply to comment #73)
> Why did you remove the SVG version from SVN?:
> 
> $ svn up
> D    data/icons/hicolor/22x22/actions/internet-radio-new.svg
> U    data/icons/hicolor/22x22/actions/Makefile.am
> A    data/icons/hicolor/22x22/actions/internet-radio-new.xcf
> U    data/icons/hicolor/22x22/actions/internet-radio-new.png

It was the source for the "old" icon, new source is internet-radio-new.xcf.

> This breaks the compilation:

Yeah, sorry, I forgot to change the Makefile.am.
Yet fixed on svn.
Comment 75 Alex Lancaster 2007-06-04 11:52:59 UTC
(In reply to comment #74)
> (In reply to comment #73)
> > Why did you remove the SVG version from SVN?:
> > 
> > $ svn up
> > D    data/icons/hicolor/22x22/actions/internet-radio-new.svg
> > U    data/icons/hicolor/22x22/actions/Makefile.am
> > A    data/icons/hicolor/22x22/actions/internet-radio-new.xcf
> > U    data/icons/hicolor/22x22/actions/internet-radio-new.png
> 
> It was the source for the "old" icon, new source is internet-radio-new.xcf.

But the SVG files are supposed to be icons in their own right and are supposed to be the way forward ultimately for scalable icons, I thought?
Comment 76 Alex Lancaster 2007-06-04 11:55:52 UTC
Hence the presence of the hicolor/scalable directory.  Shouldn't the icons be generated from the SVG source there?
Comment 77 Jonathan Matthew 2007-06-04 12:30:24 UTC
If only it was that simple..  (if it was, we'd just generate the smaller sizes at runtime)

The 'scalable' icons typically look blurry and indistinct when scaled down (but they look good scaled up), so we use either custom bitmaps or simplified SVG for the smaller sizes.  Compare 16x16/apps/rhythmbox.svg and scalable/apps/rhythmbox.svg in rsvg-view to see what I mean.  At around 16x16, the 'scalable' version is a smudge, and at larger sizes, the 16x16 version is blocky and lacks detail.
Comment 78 Michael Monreal 2007-06-07 10:26:06 UTC
Created attachment 89541 [details]
1st try of a new library icon

I don't really like the mimetype-icon used for the music library source (plus, I don't even understand where it is pulled from...). So I tried to come up with something better. I tried to do a shelf of CDs but this seems to be near-impossible at the required size... Then I came up with this. It's not executed very well but what do you think of the basic idea?
Comment 79 Luca Ferretti 2007-06-10 12:39:38 UTC
I think is better split this bug, using one bug report for each missing/needed icon, adding new bugs here as "depends on"

So, I've just opened:
  * bug 440608 - icon for Browser button
  * bug 440609 - icon for Library entry
  * bug 440611 - icon for shared music entries

doh, it seems I can't edit the depends on list :-(
Comment 80 Jonathan Matthew 2007-06-23 06:21:23 UTC
Created attachment 90508 [details] [review]
make magnatune and jamendo icons themeable

magnatune_circle_small.png -> plugins/magnatune/icons/hicolor/24x24/places/magnatune.png

jamendo_logo_small.png -> plugins/jamendo/icons/hicolor/24x24/places/jamendo.png

These are then installed under $pkgdatadir with the rest of our icons.

Also adds some ugly magic in the rb module to add the plugin source path to the icon search path for uninstalled builds.
Comment 81 Jonathan Matthew 2007-10-24 14:29:41 UTC
I've committed that because it's probably as good as we can do.

What's left?
Comment 82 Jones Lee 2008-05-09 01:11:55 UTC
Only left the Browser icon, it'd be reasonable to remove the Hide/Show Browser toggle button since there are no sane metaphor for this icon. Can someone cook a patch?
Comment 83 Jonathan Matthew 2008-05-09 01:18:43 UTC
You want us to remove the button because you can't think of an icon to display on it?  Uh.. no.
Comment 84 Jones Lee 2008-05-09 03:39:50 UTC
Well, sorry if I accidentally say something wrong or really bad. IMHO, based on trials of many other music software on Linux platform, I still think it's the best to remove that button. However, if you do not share my same humble opinion, my apology and I hope some artist can cook an icon for this button. CC jimmac, lapo, andreasn or hylke.
Comment 85 Michael Monreal 2008-05-09 08:07:30 UTC
(In reply to comment #83)
> You want us to remove the button because you can't think of an icon to display
> on it?  Uh.. no.

Uhm, yes. This is exactly what UI design is about. You don't know how to do it right (even after over a year!) so doing it was obviously wrong. 

Comment 86 Jonathan Matthew 2008-05-10 01:35:49 UTC
Simply removing things that aren't perfect is *not* what UI design is about.  I believe someone was working on moving the browser toggle somewhere else (can't be bothered chasing up a bug number right now); perhaps you'd have some constructive criticism on that?
Comment 87 Jones Lee 2008-05-10 04:54:18 UTC
(In reply to comment #86)
> Simply removing things that aren't perfect is *not* what UI design is about.  I
> believe someone was working on moving the browser toggle somewhere else (can't
> be bothered chasing up a bug number right now); perhaps you'd have some
> constructive criticism on that?
> 

You refer to http://bugzilla.gnome.org/show_bug.cgi?id=366499  ?
Comment 88 Karel Demeyer 2008-05-10 12:50:28 UTC
IMHO, I think this option isn't something a user will/should change a lot. Therefor it's something that should be set in the preferences panes (which you only open if you want to make changes about how you use the application).  
Menu-items and toolbar-buttons should be there for options/functionality a typical user changes during the usage of the program... those are the tools that make part of the program.  Great examples are a play button, previous/next, or even a button to show lyrics (most ppl, I think, only want this for some songs but it makes part of the functionality).  
Hiding/showing the browser is, IMO, a way of using the program.  It's like choosing where to store you music library ... You chose this once according to your likings but then you probably don't want to change this behaviour very often.
If the browser is something ppl turn on when they want to quicly find a song and then turn if off again, I think it means there's something wrong with the browser itself as it shouldn't be in the way of the user who needs it to find songs.
(This in only *my* UI design interpretation)
Comment 89 Jones Lee 2008-05-10 13:40:35 UTC
Everyone has their own opinions on usability, I think we just should respect the GUI designer of the program. The compromising solution is to have a customize toolbar feature like EOG or Evince in which DnD toolbar icon is possible. Then everyone can customize the GUI to their own taste.

Let's design an icon for Browser and end this going-no-where discussion.
Comment 90 Luca Ferretti 2008-06-15 15:02:26 UTC
IMHO a good icon metaphor for browser is something like the one in KDE4 Dolphin "Columns" toolbar button. See [1]

We can't produce a good metaphor of browser concempt, so let the icon represent the UI appearance: smart, isn't it?

[1] http://www.kde.org/announcements/visual_guide_images-4.0-alpha1/dolphin.png
Comment 91 GNOME Infrastructure Team 2018-05-24 12:09:48 UTC
-- 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/280.