notify-osd needs icons in GNOME default otherwise volume changes are hidden

Bug #331383 reported by Taylor "Ripps" LeMasurier-Wren
90
This bug affects 10 people
Affects Status Importance Assigned to Milestone
notify-osd (Ubuntu)
Fix Released
High
David Barth

Bug Description

osd-notify uses its own icon namespace. There are Canonical icons in the Human theme those those slots, but not yet in the GNOME default theme. The icons in the Human theme should not be used in the GNOME default theme, separate GNOME-default-style icons should be placed in the GNOME-default theme, and licensed appropriately.

Symptom description:
Apparently notify-osd takes over compiz's volume osd, but the problem is that it doesn't work. When I used the volume control buttons on my keyboard, all I see is a empty black box in the top-left corner of the screen. I'm currently unable to visually tell what the volume is.

Revision history for this message
Mirco Müller (macslow) wrote :

There are some issues with the related patch to gnome-settings-daemon and the human-icon-theme >= 0.33. We'll look into it.

Changed in notify-osd:
assignee: nobody → macslow
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Ivan Stetsenko (stetzen) wrote :

This also affects the indicator of changing brightness.

Revision history for this message
Thiago Teixeira (tvst) wrote :

"There are some issues with the related patch to gnome-settings-daemon and the human-icon-theme >= 0.33. We'll look into it."

I'm having the same problem, but I use the Gnome icon theme, not Human. So I don't think that's the problem.

Also, just to be clear what I'm seeing: when I press the volume key on my keyboard, I get the dark translucent OSD rectangle on the top-right of the screen -- but there is nothing inside it. No text. No icons. Nothing.

Revision history for this message
Taylor "Ripps" LeMasurier-Wren (ripps818) wrote :

I fixed the problem in mine by making sure my icon theme inherited from the Human Icon theme in it's index.theme file.
Apparently notify-osd uses it's own icon naming scheme that only Human has.

Revision history for this message
Martin Lettner (m.lettner) wrote :

I can confirm this bug (notifications like pidgin or battery work perfectly but volume and brightness changes just show the empty black translucent box) and I can also confirm the workaround with inheriting the Human icon theme when using another icon theme.

Revision history for this message
Taylor "Ripps" LeMasurier-Wren (ripps818) wrote :

Most icons themes fallback to gnome when missing icons, not Human. The notification icons should be placed in the gnome icon directory as well as the Human.

Revision history for this message
Mark Shuttleworth (sabdfl) wrote : Re: [Dx-team] [Bug 331383] Re: notify doesn't show volume changes

We will provide a set of icons in the Gnome theme as well. I'll update
this bug to reflect that.

description: updated
David Barth (dbarth)
Changed in notify-osd:
assignee: macslow → dbarth
Revision history for this message
Michaël Vanderheeren (michael-vanderheeren) wrote :

Neither way, I think introducing the notification-* prefix in the icon packs is good, but it makes other things look inconsistent. Maybe someone should be thinking about cleaning up the names.

Example: you now use notification-*. There are still a lot of files with nm-* (network manager). Either way then it should be ntf-* (or something like that for notification) or what would be better is using networkmanager-* which makes icon sets more clear.

On the other hand there should always be a fall-back for those notifications. It should not depends on a theme i think... Maybe there should be at least a textual fall-back.

At last, the icons should indeed be included in the Gnome theme as Mark Shuttleworth stated above. I heard rumors about adding them to the high-color theme because every theme inherits from that theme. That should NOT be done. It is called for a reason high-color and the icons currently used for notify-osd are NOT high-color icons.

Revision history for this message
Kenneth Wimer (kwwii) wrote :

We have named these in accordance with the freedesktop icon naming spec. Yes there can and will be some improvements to the names.

Hicolor is a place for apps to put their icons and is the last fallback in the icon hierarchy. It has nothing to do with the actual look of the icons.

Revision history for this message
Andreas Nilsson (andreasn) wrote :

(I think I already mentioned this in a similar bug report about this that I can't find right now, so sorry if I'm repeating myself)

Note that these icons shouldn't be placed in /usr/share/icons/hicolor/... , but rather /usr/share/notify-osd/icons/hicolor/... (similar to what f-spot, banshee, gpm etc. do) as /usr/share/icons/hicolor tend to be reserved for app launcher icons only. See more info here: http://live.gnome.org/ThemableAppSpecificIcons/

Same thing applies to networkmanager actually, but we haven't fixed that quite yet. I guess we'll try to get that fixed when we take care of bgo #504822.

Revision history for this message
Kenneth Wimer (kwwii) wrote :

Right, the fixes are already commited, dbarth would know when they will make it into the wild

Revision history for this message
David Barth (dbarth) wrote :

released as of notify-osd-0.9

Changed in notify-osd:
status: Confirmed → Fix Released
Revision history for this message
Taylor "Ripps" LeMasurier-Wren (ripps818) wrote :

This issue hasn't been resolved for me. I have notify-osd-0.9 installed. I had to put Human back in my icon theme as a inherit in order to get Volume OSD to work.
Also, adding hicolor as an inherit didn't fix it either.

Revision history for this message
Kenneth Wimer (kwwii) wrote :

Check /usr/share/notify-osd/icons/hicolor/scalable/status/ and see if the notification-audio-* icons exist. Quite a few apps put their icons in a similar path so it should work.

Revision history for this message
Taylor "Ripps" LeMasurier-Wren (ripps818) wrote :

Close it's in /usr/share/notify_osd/... (notice it's not a dash). Maybe that's the problem.

Revision history for this message
Filippo Argiolas (fargiolas) wrote :

sudo mv /usr/share/notify_osd /usr/share/notify-osd/ fixes the issue here...

Revision history for this message
Filippo Argiolas (fargiolas) wrote :

Well, I still miss brightness icons (and brightness notification)

Revision history for this message
Kenneth Wimer (kwwii) wrote :

David?

affects: notify-osd → notify-osd (Ubuntu)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.