[Xfce4-commits] CVS: xfce4/xfdesktop/settings backdrop_settings.c, 1.84, 1.85 backdrop-icon.h, 1.7, NONE menu.png, 1.1, NONE

[Benedikt Meurer]

Brian Tarricone wrote:
>      mcs_plugin->caption = g_strdup (_("Desktop"));
>      mcs_plugin->run_dialog = run_dialog;
>  
> -    mcs_plugin->icon = gdk_pixbuf_new_from_inline(-1, backdrop_icon_data,
> -			FALSE, NULL);
> +    mcs_plugin->icon = xfce_themed_icon_load("xfce4-backdrop", 32);

Don't you think its better to have all mcs icons at the same size? The 
desktop plugin is the only one to use a 32x32 icon, which looks kinda 
odd. We should either change all other plugins to use 32x32 or the 
desktop plugin should use the 48x48 size as well.

Benedikt