-
-
Notifications
You must be signed in to change notification settings - Fork 578
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing icon and app name in the dash of Gnome Shell #292
Comments
Making some researches I found out these:
TLDR: The app needs an |
Maybe the |
Mm, it's not a problem related to iced. The icon displays just fine in the Debian based and Red Hat based distros (see here the corresponding files: https://github.com/GyulyVGC/sniffnet/tree/main/resources/packaging/linux). I didn't personally packages it for Arch Linux. |
Did you test on Gnome Wayland? |
Look, I replaced the content of this desktop file (the custom Arch one uses the keyword key and different categories). Even tried to reboot the system, but the icon is still not showing. I don't think the package is built in a different way you normally do. You can see here: https://gitlab.archlinux.org/archlinux/packaging/packages/sniffnet/-/blob/main/PKGBUILD#L26-43 I don't know what this post install is doing: https://gitlab.archlinux.org/archlinux/packaging/packages/sniffnet/-/blob/main/sniffnet.install
But I don't think it's related to the menu. I can confirm, at least on Gnome Wayland 44, the dash can't see the name of the application and its icon. |
Nope
It has nothing to do with icons, that script just sets the capabilities to inspect a network adapter.
I'm not able to try the app on Gnome Wayland right now, so I cannot troubleshoot it at the moment. |
As I pointed out, this issue could be resolved setting an Unfortunately I'm not expert with Rust, otherwise I would have tried to test the library. Anyway, can you try to set an app id? Should be the id property here of the Settings structure attached to the window instance: https://docs.rs/iced/latest/iced/settings/struct.Settings.html Winit has this method to set it: https://github.com/elinorbgr/winit/blob/5643865387d805ae7557af2c491deafe11ce97fa/src/os/unix.rs#L229-L234 Should be something similar for iced. |
Sure, I will set that field to "sniffnet". |
Is there a way to test the application without installing the build files under Arch package is built with |
Done. |
Wow, thank you so much for having let me know about that id field 🥇 |
@all-contributors please add @Digitalone1 for bug. |
I've put up a pull request to add @Digitalone1! 🎉 |
Hey, I'm a bit late to the party but I guess the problem is resolved with the mentioned commit. Either way, let me know if you need anything else on the Arch packaging side! |
No worries, the problem is solved and was not due to the package. |
I filed a bug report on Arch. Please use the desktop file reported here. It's safer and it uses the |
Oh, I didn't get that with the original Arch .desktop and the app id property set it still don't work |
I'm using the desktop file reported here and it works with the latest master. Don't know for the Arch one but the wm class should be specified anyway. I noticed for other applications that if it does not match the app id, the name is not shown on the Dash. |
That's because you have to run that post installation script (or run in sudo mode) |
Ok, I got it. |
Updated the Arch package with suggested changes, thanks! |
I'm afraid this has to be reopened because I restarted to use sniffnet, but I don't see the proper app icon on the Gnome Dash. I remember that it was correctly visible after setting the id in the code, but now it disappeared again. Maybe another iced issue, I don't know. Unfortunately I returned to use |
It could be related to the migration to iced 0.10. They introduced a further configuration for the app id and I think that this could be solution: iced-rs/iced#1963 |
I've just pushed the fix to the |
Thanks for you effort. So I tried two ways:
So, in order to see the icon, the build should have the Thanks @GyulyVGC |
Perfect, no problem! Ps: I've also started adding advanced settings in a secondary branch and soon an option to change the report location will be included! (To be released in v1.3 as already discussed) |
Is there an existing issue for this?
What's the problem?
As shown in the screenshot. Usually this issue is resolved specifying
StartupWMClass
in the .desktop file. Tried it, but nothing changes.Maybe an issue related to iced library. I don't know. Should be investigated.
How did you install the app?
downloaded one of the provided packages
Operating System
Linux
Additional context
https://archlinux.org/packages/extra/x86_64/sniffnet/
The text was updated successfully, but these errors were encountered: