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 782268 - gnome-todo window has no icon
gnome-todo window has no icon
Status: RESOLVED FIXED
Product: gnome-todo
Classification: Other
Component: User Interface
unspecified
Other Linux
: Normal minor
: ---
Assigned To: GNOME To Do maintainer(s)
GNOME To Do maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-05-06 15:17 UTC by larchunix
Modified: 2017-05-10 22:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix proposal (756 bytes, patch)
2017-05-06 15:18 UTC, larchunix
committed Details | Review
Issue screenshot (1.37 MB, image/png)
2017-05-09 22:02 UTC, larchunix
  Details

Description larchunix 2017-05-06 15:17:08 UTC
Gnome Todo window has no icon.

This deteriorates the integration with other desktop components relying on window icons like pagers, windows task list, ...
Comment 1 larchunix 2017-05-06 15:18:50 UTC
Created attachment 351262 [details] [review]
Fix proposal
Comment 2 Georges Basile Stavracas Neto 2017-05-09 20:07:22 UTC
Review of attachment 351262 [details] [review]:

Can you please confirm why is this issue happening? GTK+ picks the application id as the icon, doesn't it?
Comment 3 larchunix 2017-05-09 22:01:34 UTC
I cannot explain why but in my case GTK+ does not seem to pick any icon.

I made some tests using the GTK inspector: the icon-name property of the GtdWindow object is not set. However, when I set it to "org.gnome.Todo" from the inspector, the icon appear.

Maybe the issue is caused by the fact that I am using gnome-flashback and metacity instead of gnome-shell and mutter.

Please find attached to this report a screenshot describing the issue.
Comment 4 larchunix 2017-05-09 22:02:50 UTC
Created attachment 351486 [details]
Issue screenshot
Comment 5 Georges Basile Stavracas Neto 2017-05-10 20:28:21 UTC
(In reply to larchunix from comment #3)

> Please find attached to this report a screenshot describing the issue.

I understand your problem. What I'm trying to figure out is whether this is an issue in GNOME To Do, or in GTK+ itself. Some questions:

 - Is there any other app that misbehaves like this?
 - On apps that work (e.g. Control Center?), is there a call to 'gtk_window_set_default_icon_name()'?
 - Which DE are you using? (Sorry, couldn't figure out from the screenshot)
Comment 6 larchunix 2017-05-10 21:48:57 UTC
(In reply to Georges Basile Stavracas Neto from comment #5)
> 
> I understand your problem. What I'm trying to figure out is whether this is
> an issue in GNOME To Do, or in GTK+ itself. Some questions:
> 
>  - Is there any other app that misbehaves like this?

I checked most of the GNOME applications installed on my computer; GNOME To Do appears to be the only one having this problem. 

>  - On apps that work (e.g. Control Center?), is there a call to
> 'gtk_window_set_default_icon_name()'?

It seems so (at least for the applications I tested):

● Cheese:
  └─ https://git.gnome.org/browse/cheese/tree/src/cheese-application.vala#n114
     └─ Window.set_default_icon_name ("org.gnome.Cheese");
● Evince
  └─ https://git.gnome.org/browse/evince/tree/shell/main.c#n299
    └─ gtk_window_set_default_icon_name ("evince");
● Gedit
  └─ https://git.gnome.org/browse/gedit/tree/gedit/gedit-app.c#n1444
     └─ gtk_window_set_default_icon_name ("gedit");
● GNOME 2048
  └─ https://git.gnome.org/browse/gnome-2048/tree/src/application.vala#n90
     └─ Window.set_default_icon_name ("gnome-2048");
● GNOME Boxes
  └─ https://git.gnome.org/browse/gnome-boxes/tree/src/app-window.vala#n138
     └─ Gtk.Window.set_default_icon_name ("gnome-boxes");
● GNOME Builder
  └─ https://git.gnome.org/browse/gnome-builder/tree/libide/application/ide-application.c#n550
     └─ gtk_window_set_default_icon_name ("org.gnome.Builder");
● GNOME Disks Utility
  └─ https://git.gnome.org/browse/gnome-disk-utility/tree/src/disks/gduwindow.c#n1021
     └─ gtk_window_set_icon_name (GTK_WINDOW (window), "gnome-disks"); 
● GNOME Power Statistics 
  └─ https://git.gnome.org/browse/gnome-power-manager/tree/src/gpm-statistics.c#n1731 
     └─ gtk_window_set_default_icon_name ("gnome-power-manager"); 
● GNOME Terminal
  └─ https://git.gnome.org/browse/gnome-terminal/tree/src/terminal-app.c#n423
     └─ gtk_window_set_default_icon_name (GNOME_TERMINAL_ICON_NAME);
● Nautilus
  └─ https://git.gnome.org/browse/nautilus/tree/src/nautilus-application.c#n1231
     └─ gtk_window_set_default_icon_name ("org.gnome.Nautilus");

>  - Which DE are you using? (Sorry, couldn't figure out from the screenshot)

Lately I have been using GNOME Flashback (formerly GNOME Fallback).
Comment 7 Georges Basile Stavracas Neto 2017-05-10 22:36:25 UTC
Perfect, thanks for the extensive research!

Attachment 351262 [details] pushed as b3d44e57 - application: set default icon name