Summary: | AppIndicator GTK apps fail to find icons in custom GTK icon theme folder and crash | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | André Brait <andrebrait> |
Component: | System Tray widget | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | andrebrait, kde, notmart, notuxius |
Priority: | NOR | ||
Version: | 5.8.2 | ||
Target Milestone: | 1.0 | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | application runnin showing the correct icon |
Description
André Brait
2016-10-24 13:28:03 UTC
>- KDE 5.8 now demands custom icon folders have their names ended in "icons". It always had. The path should match a valid icon directory as specified in https://standards.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html i.e yourpath/icons/hi-color/48x48/blah.png you would have been getting this warning before, just maybe silently working. Though we shouldn't have changed behaviour in minor releases (In reply to David Edmundson from comment #1) > >- KDE 5.8 now demands custom icon folders have their names ended in "icons". > > It always had. > The path should match a valid icon directory as specified in > https://standards.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html > > i.e yourpath/icons/hi-color/48x48/blah.png > > you would have been getting this warning before, just maybe silently working. > > > Though we shouldn't have changed behaviour in minor releases Oh, my bad then. Yeah, it used to work ok, but I don't recall getting the warning. It still works in other DE's, I think they don't check this stuff. Enforcing correctness is ok. The change was apparently somewhere between 5.6 and 5.8. Are these considered to be minor releases? do you have an application that can reproduce the problem that doesn't require any particular hardware? (In reply to Marco Martin from comment #3) > do you have an application that can reproduce the problem that doesn't > require any particular hardware? I have modified one of mine to behave like it used to when using the default options, and also to not depend on any specific hardware: https://github.com/andrebrait/nvidia-power-indicator-fake If you change the icon option to 'color', however, it sets the full icon to the icon file and it works ok. It seems that it now appears, but no icon is displayed. I can't get the same message as the user got, but I'm on Arch Linux, so many things have changed between when this bug was reported and now. At the time, the user was running KDE Neon. Please let me know if I can be of any help. Full >>>path<<< to the icon file, sorry for the typo. Created attachment 102104 [details]
application runnin showing the correct icon
Thanks.
I tried your application on master, but seems to work fine.
It starts good, the nvidia icon is shown, if i click "force nvidia off" the icon gets replaced by the intel logo as expected.
On distro packages (5.8.0) no crash happens, but the icon is broken
so would need a bit of testing to make sure it's really ok in master and current 5.8 branch
The app icons are looking ok in: Plasma: 5.12.2 Apps: 17.12.2 Frameworks: 5.43.0 Qt: 5.10.1 Kernel: 4.14.20-2-MANJARO OS: Netrunner Rolling fcitx-4.2.9.6-1 Only open part of this bug was about 5.8, and that's not getting any changes for this. |