Closed dyskette closed 7 years ago
I cannot test this under wayland session. I don't have wayland installation and Virtual Box driver doesn't support wayland. But it DOES work under Gnome's x-session.
win.set_wmclass(_("Easy eBook Viewer"), _("Easy eBook Viewer")
in main.py, that should be respected by GNOME, I'm not sure why it wouldn't.Ubuntu GNOME 16.10, GNOME 3.20.
But I also tested on Gnome 3.22 some time ago, I just did a test for any regressions. If it's wayland related I might need some help.
It works under X. Sorry for not stating that in the first comment.
I updated the icon cache manually right now just in case, log out and in, still the same. Did it with gtk-update-icon-cache
instead, because update-icon-caches
doesn't exist in Fedora.
Probably that python thing does something directly with X and Wayland is not aware of it.
I'm launching it from a .desktop
file in /usr/share/applications/easy-ebook-viewer.desktop
.
Installed with an rpm package I did myself. The .spec
file does execute the make script and install the .desktop
file included in the repo.
I just updated to the latest commit, and the issue persists.
Edit: Probably this?
Thanks for the info. I have few ideas but until I can get my hands on wayland install I won't be able to do much. As far as setting wm_class goes documentation can say whatever it wants but it was how I solved this issue for GNOME (under X server) in the first place. Try to edit src/main.py and remove that line, see if it helps.
The only other idea I have is that I'm just showing Gtk.ApplicationWindow and not making Gtk.Application object where you normally pass application_id to constructor. It really shouldn't matter but hey, that is the only thing I know that the app doesn't do by the book. I should change that anyway so I might as well do it blindly hoping it might solve this by some chance. Expect update tomorrow or day after.
Please tell me if that changes anything. I'm bit busy right now but I can try more things in few hours.
Error persists: (GNOME 3.22 on Fedora 25 with Wayland)
Edit: Maybe is a GDK issue.
Maybe is a GDK issue.
Might be. What I'm really surprised about is that even your doc doesn't display icon properly. Gnome top bar integration is one thing but displaying icon specified with launched desktop file exec cmd should be no brainier really. Like the hell?
I found something.
The bug report has a lot of discussion around it.
Perfect find. Update will be out tonight.
I hope this is it. Can you confirm?
No. Still the same. I have read the bug report comments but in the end they made this change It seems that the mailing list thread I posted earlier was created before this commit.
Also, they pointed to this wiki article.
I noticed this error in the command line:
(main.py:7885): GLib-GIO-CRITICAL **: g_application_set_application_id: assertion 'application_id == NULL || g_application_id_is_valid (application_id)' failed
Can confirm, #21 works on Wayland and X. Commits 72850eb and 7a64c63 can be reverted and the .desktop file is still associated.
@mardojai Good job.
I'll see about those commits as there still seems to be some merit in those changes.
Setting application_id
or prgname
should be enough to be paired with the desktop file. And gtk_window_set_wmclass
is deprecated anyway.
Opening it in a GNOME wayland session just shows an "unknown" icon and "main.py" as label in the task switcher.
This is GNOME 3.22 on Fedora 25.