elementary OS

Notify OSD icons don't match

Bug #1014811 reported by Cassidy James Blaede
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
elementary Icons
Fix Released
Low
Unassigned
elementary Icons 3.2 "3.2"
elementary OS
Fix Released
Low
Cody Garver
elementary OS luna-rc1 "Luna Final"

Bug Description

The icons in Notify OSD don't match the icon theme whatsoever.

Revision history for this message
Cassidy James Blaede (cassidyjames) wrote :

Check out the Wi-Fi and Volume icons on the panel compared with in the notification.

Cassidy James Blaede (cassidyjames)
Changed in elementaryicons:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Cody Garver (codygarver) wrote :

These icons are installed by notify osd.

Changed in elementaryicons:
status: Confirmed → Invalid
Changed in elementaryos:
importance: Undecided → Low
Revision history for this message
Cassidy James Blaede (cassidyjames) wrote :

Okay, then if possible let's symlink those icons to the proper ones in the theme. Or update our Notify-OSD patch to make them pull those from the icon theme.

Revision history for this message
Cassidy James Blaede (cassidyjames) wrote :

The directory structure of /usr/share/notify-osd/icons looks like it's intended to emulate the /usr/share/icons/ directory structure. Odd.

The icons we want them to end up using are in /usr/share/icons/elementary/status/symbolic/

Revision history for this message
Cassidy James Blaede (cassidyjames) wrote :

Ah, I think I figured out the proper way to fix this. The notify-osd-icons package is what provides the icons. We should make a new one that is symlinked to the proper icons in the elementary theme, or make our own package that includes the icons themselves.

http://packages.ubuntu.com/precise/notify-osd-icons

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Try copying or symlinking the folder to notify-osd's icon folder. I suspect notify-osd doesn't support symbolic icons though.

Revision history for this message
Cody Garver (codygarver) wrote :

In the course of fixing this bug for elementary I ported the chameleonic revert patch to work with newer os-patches but encountered new errors after that. Once the os-patch branch builds with this patch we can swap out the icons and close this bug.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

It seeems we won't have to patch anything, we just have to add a folder called "elementary" in /usr/share/notify-osd/icons/ and place our icons there in a layout similar to the already existing "gnome" and "hicolor" folders there.

Revision history for this message
Cody Garver (codygarver) wrote :

I have attached a branch that installs an elementary theme to the desired location. It needs someone to replace my placeholder text files with the proper icons then this bug can be closed.

no longer affects: elementaryicons
Changed in notify-osd (Ubuntu):
status: New → Invalid
Changed in elementaryos:
milestone: none → luna-beta3
status: New → In Progress
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Looks like the OS dev part here is done.

Changed in elementaryos:
assignee: nobody → elementary UX Team (elementary-design)
Cody Garver (codygarver)
Changed in elementaryos:
assignee: elementary UX Team (elementary-design) → Cody Garver (codygarver)
status: In Progress → Fix Released
Cassidy James Blaede (cassidyjames)
Changed in elementaryicons:
importance: Undecided → Low
Cody Garver (codygarver)
affects: notify-osd (Ubuntu) → noise
no longer affects: noise
Danielle Foré (danrabbit)
Changed in elementaryicons:
status: New → In Progress
Cody Garver (codygarver)
Changed in elementaryos:
milestone: luna-beta3 → luna-rc1
Revision history for this message
Danielle Foré (danrabbit) wrote :

Since we're using gala notifications now, I'm going to consider this fixed

Changed in elementaryicons:
status: In Progress → Fix Committed
milestone: none → 3.2
Danielle Foré (danrabbit)
Changed in elementaryicons:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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