affinity applet icon size does not change

Bug #205413 reported by reacocard
6
Affects Status Importance Assigned to Milestone
Awn Extras
Fix Released
Low
moonbeam

Bug Description

The icon for the affinity applet does not change size properly (screenshot: http://ubuntuforums.org/attachment.php?attachmentid=63473&d=1206229445)

this is using reacocard's awn-bzr PPA packages, bzr207.

reacocard (reacocard)
Changed in awn-extras:
status: New → Confirmed
Revision history for this message
reacocard (reacocard) wrote :
Revision history for this message
Andrew Starr-Bochicchio (andrewsomething) wrote :

I'll leave this set a confirmed, but I can not reproduce this with awn-extras built from source. I notice both those screen shots seem to be using the human icon theme. Can any one confirm this using a different theme?

Changed in awn-extras:
importance: Undecided → Low
Revision history for this message
dje (duncaneastoe) wrote :

Screenshots posted are mine, I have changed theme to Clearlooks and the icon is the correct size. Please see the attached screenshot

Revision history for this message
moonbeam (rcryderman) wrote :

Looks to me like gtk_icon_theme_load_icon () isn't scaling the icon for some reason.

See:
http://library.gnome.org/devel/gtk/2.12/GtkIconTheme.html#gtk-icon-theme-load-icon
and
http://library.gnome.org/devel/gtk/2.12/GtkIconTheme.html#gtk-icon-info-load-icon

My guess is that gtk_icon_theme_load_icon () is only able to find a small "search" icon and "GTK+ will avoid scaling icons that it considers sufficiently close to the requested size or for which the source image would have to be scaled up too far"

I've pushed a committed a change that uses gdk_pixbuf_scale_simple() to scale the icon if that provided by gtk_icon_theme_load_icon () isn't within 10% of the size requested. Please test.

Rev: 376

Changed in awn-extras:
status: Confirmed → In Progress
assignee: nobody → rcryderman
Revision history for this message
dje (duncaneastoe) wrote :

Did an apt-get upgrade and awn was upgraded but there is no change

Revision history for this message
moonbeam (rcryderman) wrote :

as far as I know this is not in any binary packages yet. At this point it would need to be build from source.

Revision history for this message
reacocard (reacocard) wrote :

The hardy packages in my PPA now have this fix, and I can confirm it works.

Revision history for this message
dje (duncaneastoe) wrote :

I can also confirm it is working

moonbeam (rcryderman)
Changed in awn-extras:
status: In Progress → Fix Committed
Mark Lee (malept)
Changed in awn-extras:
milestone: none → 0.2.8
moonbeam (rcryderman)
Changed in awn-extras:
status: Fix Committed → Fix Released
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.