! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Feature request: Make the plugin themable.
Status:
RESOLVED: FIXED
Severity:
enhancement
Product:
Xfce4-weather-plugin
Component:
General

Comments

Description TAXI 2010-05-21 17:51:15 CEST
The plugin uses his own theme directory so changing the weather icons is only possible by replacing them.
I wrote a quick&dirty patch who changes this. The icons will be loadet via the xfce_themed_icon_load method and the filename was changed to xfce4-weather-XY (from: XY)

P.S. Sorry for my bad english :)
Comment 1 TAXI 2010-05-21 17:52:02 CEST
Created attachment 2988 
weather-themable.patch
Comment 2 TAXI 2010-05-21 19:42:24 CEST
Created attachment 2989 
weather-themable2.patch

I played a bit and here is a new patch: It does the same as bevore but fixes the Makefile(s) and the icon-names. I can use this patch in the build process of gentoo linux and it works. :)
Comment 3 TAXI 2010-05-24 13:15:57 CEST
Created attachment 2992 
new-weather-themable.patch

Patch rewritten (less changes in panel-plugin/weather-icon.c)
Comment 4 TAXI 2010-05-24 14:47:22 CEST
Created attachment 2993 
weather-themable-rev4.patch

Even more simplyfied (use g_strdup_printf and not strcpy/strcat)
Comment 5 TAXI 2010-05-25 15:43:30 CEST
Created attachment 2996 
weather-themable-rev5.patch

fixed an error in the configure script
Comment 6 Harald Judt 2012-07-17 13:53:42 CEST
Long time ago. Don't know how much of it is still usable after all the changes, and what's already in git. Let's look into it after the next release.
Comment 7 Harald Judt 2012-11-10 11:03:45 CET
With http://git.xfce.org/panel-plugins/xfce4-weather-plugin/commit/?id=01787dffe9470ef806d6b0f5089127de316ef5ed, the plugin finally supports loading a custom theme (but not fdo-standardized themes, and probably never will) from an arbitrary directory. Automatic theme detection and GUI elements for easy theme selection are still missing but planned for the new configuration dialog which will be available in upcoming 0.8.3. I'm closing this bug now before it starts to rot ;-)

Bug #6464

Reported by:
TAXI
Reported on: 2010-05-21
Last modified on: 2012-11-10

People

Assignee:
Harald Judt
CC List:
1 user

Version

Version:
0.7.3 or older

Attachments

Additional information