Merge lp:~ilkka/gwibber/gtk-find-icon-loading-fix into lp:gwibber

Proposed by Ilkka Laukkanen
Status: Merged
Merge reported by: Ryan Paul
Merged at revision: not available
Proposed branch: lp:~ilkka/gwibber/gtk-find-icon-loading-fix
Merge into: lp:gwibber
Diff against target: 23 lines
1 file modified
gwibber/gwui.py (+3/-3)
To merge this branch: bzr merge lp:~ilkka/gwibber/gtk-find-icon-loading-fix
Reviewer Review Type Date Requested Status
gwibber-committers Pending
Review via email: mp+14011@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ilkka Laukkanen (ilkka) wrote :

This is a fix for #427249. By using the builtin gtk-find icon of the default icon theme, a crash is avoided in Karmic's KDE.

Revision history for this message
Ilkka Laukkanen (ilkka) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'gwibber/gwui.py'
2--- gwibber/gwui.py 2009-10-22 11:43:06 +0000
3+++ gwibber/gwui.py 2009-10-27 06:55:17 +0000
4@@ -158,8 +158,8 @@
5 self.append(None, [n.capitalize(), n, None, None, icon, None, None])
6
7 for acct in self.accounts: self.add_account(acct)
8-
9- icon = pbfile(resources.icon("gtk-find", use_theme=False))
10+
11+ icon = gtk.icon_theme_get_default().lookup_icon("gtk-find", gtk.ICON_SIZE_SMALL_TOOLBAR, gtk.ICON_LOOKUP_USE_BUILTIN).get_builtin_pixbuf()
12 self.search_item = self.append(None, ["Search", "search", None, None, icon, None, None])
13
14 def add_errors(self):
15@@ -169,7 +169,7 @@
16
17 def add_search(self, query, feature="search"):
18 pbfile = gtk.gdk.pixbuf_new_from_file
19- icon = pbfile(resources.icon("gtk-find", use_theme=False))
20+ icon = gtk.icon_theme_get_default().lookup_icon("gtk-find", gtk.ICON_SIZE_SMALL_TOOLBAR, gtk.ICON_LOOKUP_USE_BUILTIN).get_builtin_pixbuf()
21 path = "/gwibber/%s/%s" % (feature, query)
22 opid = "search-%s" % query
23 ti = self.append(self.search_item, [query, path, None, feature, icon, True, opid])