"Suspend" icon is black rectangle in "Shut Down" dialog

Confirmed on Intrepid alpha 5, all updates installed

The hibernate icon also seems a bit out of place... I wonder why this dialog isn't using the theme icons. For instance, human-icon-theme installs:

/usr/share/icons/Human/22x22/apps/gnome-session-suspend.png
/usr/share/icons/Human/24x24/apps/gnome-session-suspend.png
/usr/share/icons/Human/48x48/apps/gnome-session-suspend.png
/usr/share/icons/Human/scalable/apps/gnome-session-suspend.svg

It also include matching shut-down, restart, and hibernate icons in the same locations...

"Restart" icon also seems to be a lot smaller than the other icons. This breaks vertical text alignment.

"Restart" being too small is bug 269502, which I already linked to from this report.

Matthew,

I'm not sure that this is an issue with the human-icon-theme. While the issues you point out in other bug reports are resolved for me using different themes, this issue persists. The "black rectangle icon" is used with all icon themes included with a default Ubuntu Intrepid.

The problem is not with the icon theme but rather the icons that the code refers to. The small arrow is the "refresh" icon, an action. Instead it should be using the icons from the apps icons gnome-session-*.png .

OK, so the icon naming issue I won't get into. The quick and dirty way around this is to draw the view-refresh icon as well as a system-shutdown icon in the human theme.

The "suspend" icon on Quit prompt is missing in the Tango icon theme also...

Ok, I have submitted a new human icon theme package (0.30) with the necessary icons. Should be fixed in Intrepid soon. You can test it from my PPA

I've tested your new human-icon-theme package, but it still does show a black rectangle.
After some investigation I've found out the following:

The icon for the suspend-action at the moment needs to be called "sleep", take e.g. a look at
80_new_upstream_session_dialog.patch from gnome-session sources:

Here is the relevant part (gnome-session-2.24.0/gnome-session/gsm-logout-dialog.c)

+#define GSM_ICON_LOGOUT "system-log-out"
+#define GSM_ICON_SWITCH "system-users"
+#define GSM_ICON_SHUTDOWN "system-shutdown"
+#define GSM_ICON_REBOOT "view-refresh"
+/* TODO: use gpm icons? */
+#define GSM_ICON_HIBERNATE "drive-harddisk"
+#define GSM_ICON_SLEEP "sleep"

As you can see, the naming scheme is far from beeing ideal. Maybe a bug against gnome-session should be filed to use sane icon-names for this kind of actions (and which are included in gnome/tango/human themes).

gnome-session-bug opened, you'll find it here: Bug #274889

@Kenneth
I've checked out latest bzr-branch (rev 60) of human-icon-theme. I saw that you've already checked in sleep.svg and the other files, but it seems like you've forgotten to add the new icons to Makefile.am, so if you run autogen.sh/dpkg-buildpacke, a package/an install without the new icons is produced.

Yes, I realized my error after testing the package. I have added them to the make file and updated the package which I submitted for submisison. Because the version number is the same I cannot host it on my PPA (I'd have to make a different version and it didn't seem worth it). I will update the package on launchpad. A fix should be coming soon.

This bug was fixed in the package human-icon-theme - 0.30

---------------
human-icon-theme (0.30) intrepid; urgency=low

  * Reverting spelling error, it was taken care of in the scripts already and
    the package therefor failed to build.

human-icon-theme (0.29) intrepid; urgency=low

  * Adding icons to fix LP: #269504 as well as LP: #269500 .
  * Found an issue with one icon name
  * Adding a link to folder-move and folder-copy for evolution

 -- Kenneth Wimer <email address hidden> Fri, 26 Sep 2008 15:38:06 +0200