From: Christopher M. <cpm...@co...> - 2011-05-11 20:13:54
|
On 05/11/2011 03:57 PM, sd...@ta... wrote: > It doesn't break it as "will not work", but it wont work optimal. I see. So not an "actual" breakage (in that code stopped working) ... good to know :) dh The > cache is designed for use with one theme. It should be the theme designers > responsibility to add theme inherits, and not e17 responsibility to show > random icons. > > S. > >> Log: >> E17: Revert fdo icon searching in different themes. Apparently it >> breaks efreet cache (tho I saw no side effects here). >> >> >> >> Author: devilhorns >> Date: 2011-05-11 06:55:22 -0700 (Wed, 11 May 2011) >> New Revision: 59332 >> Trac: http://trac.enlightenment.org/e/changeset/59332 >> >> Modified: >> trunk/e/src/bin/e_utils.c >> >> Modified: trunk/e/src/bin/e_utils.c >> =================================================================== >> --- trunk/e/src/bin/e_utils.c 2011-05-11 12:24:13 UTC (rev 59331) >> +++ trunk/e/src/bin/e_utils.c 2011-05-11 13:55:22 UTC (rev 59332) >> @@ -452,21 +452,6 @@ >> size = e_util_icon_size_normalize(size * e_scale); >> >> path = efreet_icon_path_find(e_config->icon_theme, icon, size); >> - if (!path) >> - { >> - path = efreet_icon_path_find("default", icon, size); >> - if (!path) >> - { >> - path = efreet_icon_path_find("hicolor", icon, size); >> - if (!path) >> - { >> - path = efreet_icon_path_find("gnome", icon, size); >> - if (!path) >> - path = efreet_icon_path_find("Human", icon, size); >> - } >> - } >> - } >> - >> if (!path) return 0; >> >> e_icon_file_set(obj, path); >> |