After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 731201 - ThemedIcon icon lookup problems with 3.13
ThemedIcon icon lookup problems with 3.13
Status: RESOLVED DUPLICATE of bug 737000
Product: gtk+
Classification: Platform
Component: .General
3.13.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2014-06-04 11:24 UTC by Christoph Reiter (lazka)
Modified: 2014-09-22 07:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
example (969 bytes, application/x-python)
2014-06-04 11:24 UTC, Christoph Reiter (lazka)
Details

Description Christoph Reiter (lazka) 2014-06-04 11:24:04 UTC
Created attachment 277867 [details]
example

I create a g_themed_icon_new_from_names() with {"icon-symbolic", "icon"} as arguments.

Since 3.13 this shows a non symbolic icon while 3.12 shows the symbolic one. Only passing "icon-symbolic", makes both versions show the symbolic one.

Company pointed out on IRC that this is a known bug, but since there was no progress on fixing this last week I'm reporting it here so it doesn't get lost.

Example (Python) attached.
Comment 1 André Klapper 2014-06-04 15:50:26 UTC
Likely a regression from the changes in https://mail.gnome.org/archives/desktop-devel-list/2014-April/msg00086.html
Comment 2 Matthias Clasen 2014-06-28 05:48:01 UTC
Example program shows 2 identical icons here. I assume this is fixed, then.
Comment 3 Christoph Reiter (lazka) 2014-06-28 07:39:33 UTC
Not here, but I guess it's something with my jhbuild env then. Thanks for looking into it.
Comment 4 Christoph Reiter (lazka) 2014-09-21 18:31:56 UTC
See the same thing on debian (non jhbuild) now. It only happens if another icon theme is used (gnome-brave in my case).

The question is if ThemedIcon should search for all names in each theme separately or consider all themes for each name.
Comment 5 Matthias Clasen 2014-09-21 18:52:14 UTC
this is just more of the same: patching icon themes together by inheritance is just conceptually broken.
Comment 6 Christoph Reiter (lazka) 2014-09-21 19:15:06 UTC
I can confirm that https://bugzilla.gnome.org/show_bug.cgi?id=737000#c11 fixes this.
Comment 7 Christoph Reiter (lazka) 2014-09-22 07:40:21 UTC

*** This bug has been marked as a duplicate of bug 737000 ***