AllTray should support scalable icons (e.g., SVG)

Bug #382347 reported by Bruce van der Kooij on 2009-06-01
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
AllTray
Wishlist
Unassigned

Bug Description

Current windowing specifications (see EWMH, ICCCM) specify that each window sets an XPM icon for itself, which is stored in the X Server as a resource. AllTray currently fetches this icon through wnck's get_icon method and uses it in the tray. However, these icons are not scalable and AllTray should use scalable icons (such as SVG icon images) when possible.

There is a window manager hint called WM_ICON_NAME but that refers to the window name (see ICCCM § 4.1.2.2 which states: "The WM_ICON_NAME property is an uninterpreted string that the client wants to be displayed in association with the window when it is iconified (for example, in an icon label).", it is similar to WM_NAME).

With this information the only way to really figure out the application icon (not the window icon) is by caching all .desktop files and find the application icon by matching the values from /proc against the Exec keys. If a .desktop file cannot be found the procedure below may provide a nice fallback mechanism.

  1. Find an icon using the process name (as reported by /proc/$PID/stat{,us} on GNU/Linux or by libgtop)
  2. If that fails, use icon (or mini_icon) provided by libwnck for the application.

This bug depends on bug 382548 being implemented.

description: updated
Michael B. Trausch (mtrausch) wrote :

Confirming bug.

description: updated
summary: - Implement resize functionality for tray icons
+ AllTray should support scalable icons (e.g., SVG)
Changed in alltray:
importance: Undecided → Medium
milestone: none → 0.7.3dev
status: New → Confirmed
Changed in alltray:
milestone: 0.7.3dev → 0.7.4dev
Michael B. Trausch (mtrausch) wrote :

I don't know when this will be done... bumping it down. It'll get done when it gets done...

Changed in alltray:
importance: Medium → Wishlist
milestone: 0.7.4dev → none
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers