Opened 9 years ago

Last modified 7 weeks ago

#856 new enhancement

Allow icons to be themeable through the global icon theme

Reported by: mooch Owned by: salinasv
Milestone: 3.0.0 Component: pidgin (gtk)
Version: 2.0 Keywords:


I have difficulties to realize that i have received an IM ever since I moved from Gaim tray icons to pidgin ones.

I would like the tray icons to be themable, so that i can change them without having to override the app defaults.

Change History (12)

comment:1 Changed 9 years ago by lithium

The solution would be to move the icons from share/pixmaps/pidgin/* to a "private" hicolor theme, say, share/pidgin/icons/hicolor/$size/$category*

Pidgin then has to add this share/pidgin/icons/hicolor dir to the list of paths where it looks for icons.

What do we win here?

Well first, it would follow the standard ;)

Second, users could override icons by copying icons of the same name to their .icons/hicolor/ dir in their home directory, making the whole app themable.

comment:2 Changed 9 years ago by hbons

  • Owner set to hbons

I'm planning on doing this.

comment:3 Changed 9 years ago by hbons

  • Summary changed from Allow tray icons to be themable. to Allow icons to be themeable through the global icon theme

comment:4 Changed 9 years ago by hbons

  • Status changed from new to assigned

comment:5 Changed 9 years ago by hbons

  • Milestone set to 3.0.0

I think this is a thing for 3.0.0.

comment:6 Changed 7 years ago by QuLogic

Ticket #11031 has been marked as a duplicate of this ticket.

comment:7 Changed 7 years ago by azrael

Pidgin doesn't follow the Icon Naming Spec:

Pidgin needs to rename the icons it installs to match the names in the icon-naming-spec. Then, it should use gtk_icon_theme_append_search_path() to append its directory to the end of the path (on GTK+ 2.4 and above) to provide the default icons if the user's theme doesn't define them. Then icons should be loaded with gtk_icon_theme_lookup_icon() on GTK+ 2.4 and above. (The fallback code can just load the icons from Pidgin's own directory.)

Originally reported in Ubuntu:

comment:8 Changed 6 months ago by salinasv

There is already work in progress to get this work done in a PR. It is a partial migration at this time, but it is a good step forward.


comment:9 Changed 6 months ago by salinasv

  • Owner changed from hbons to salinasv

comment:10 Changed 4 months ago by salinasv

Related #16697

comment:12 Changed 7 weeks ago by QuLogic

Can you remove stuff from the spreadsheet that we don't use?

Note: See TracTickets for help on using tickets.
All information, including names and email addresses, entered onto this website or sent to mailing lists affiliated with this website will be public. Do not post confidential information, especially passwords!