diff options
| author | Cosimo Cecchi <cosimoc@gnome.org> | 2013-01-21 16:57:27 -0500 |
|---|---|---|
| committer | Cosimo Cecchi <cosimoc@gnome.org> | 2013-01-21 16:57:27 -0500 |
| commit | ceb631b60ff580d934792b32a0d31e7b8d1b1148 (patch) | |
| tree | 4e28827f85c947bce99fec6974f4db7211fbba65 | |
| parent | 2cf8970f91f2c81e12378119054d8a07970b53de (diff) | |
| download | gtk+-ceb631b60ff580d934792b32a0d31e7b8d1b1148.tar.gz | |
Fix a FIXME about using symbolic icons
Use g_file_info_get_symbolic_icon() for it.
| -rw-r--r-- | gtk/gtkplacessidebar.c | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/gtk/gtkplacessidebar.c b/gtk/gtkplacessidebar.c index 875bf9c44b..018eeed864 100644 --- a/gtk/gtkplacessidebar.c +++ b/gtk/gtkplacessidebar.c @@ -658,7 +658,7 @@ add_application_shortcuts (GtkPlacesSidebar *sidebar) /* FIXME: we are getting file info synchronously. We may want to do it async at some point. */ info = g_file_query_info (file, - "standard::display-name,standard::icon", + "standard::display-name,standard::symbolic-icon", G_FILE_QUERY_INFO_NONE, NULL, NULL); /* NULL-GError */ @@ -670,12 +670,7 @@ add_application_shortcuts (GtkPlacesSidebar *sidebar) GIcon *icon; name = g_file_info_get_display_name (info); - - /* FIXME: in commit 0ed400b9c1692e42498bff3c10780073ec137f63, nautilus added the ability - * to get a symbolic icon for bookmarks. We don't have that machinery. Should we - * just copy that code? - */ - icon = g_file_info_get_icon (info); + icon = g_file_info_get_symbolic_icon (info); uri = g_file_get_uri (file); tooltip = g_file_get_parse_name (file); @@ -1021,7 +1016,7 @@ update_places (GtkPlacesSidebar *sidebar) /* FIXME: we are getting file info synchronously. We may want to do it async at some point. */ info = g_file_query_info (root, - "standard::display-name,standard::icon", + "standard::display-name,standard::symbolic-icon", G_FILE_QUERY_INFO_NONE, NULL, NULL); /* NULL-GError */ @@ -1032,11 +1027,7 @@ update_places (GtkPlacesSidebar *sidebar) if (bookmark_name == NULL) bookmark_name = g_strdup (g_file_info_get_display_name (info)); - /* FIXME: in commit 0ed400b9c1692e42498bff3c10780073ec137f63, nautilus added the ability - * to get a symbolic icon for bookmarks. We don't have that machinery. Should we - * just copy that code? - */ - icon = g_file_info_get_icon (info); + icon = g_file_info_get_symbolic_icon (info); mount_uri = g_file_get_uri (root); tooltip = g_file_get_parse_name (root); |