Bug 491854 - Apps using IconThemePath are missing system tray icon
Summary: Apps using IconThemePath are missing system tray icon
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: master
Platform: Other Linux
: HI normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2024-08-18 11:19 UTC by Parag W
Modified: 2024-08-22 22:13 UTC (History)
5 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/kirigami/-/commit/5cfb3bb97213e7e5b4234143ceb5be4865be37a9
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Parag W 2024-08-18 11:19:34 UTC 
SUMMARY
Distro: Arch Linux
KDE: Built from sources (kdesrc-build)

I have JetBrains Toolbox app installed (https://www.jetbrains.com/toolbox-app/) and it runs on startup / minimizes with a Panel icon  - Recent builds (over the last week) have stopped showing its icon in the panel.

Going back to distro KDE brings the icon back.

STEPS TO REPRODUCE
1.  Install Jetbrains Toolbox app (fuse2 is a dependency) 
2. Start it 
3. Panel icon is missing

OBSERVED RESULT
Panel icon is missing/reverts to default no-icon.

EXPECTED RESULT
App icon is visible.

SOFTWARE/OS VERSIONS

Operating System: Arch Linux 
KDE Plasma Version: 6.1.80
KDE Frameworks Version: 6.6.0
Qt Version: 6.7.2
Kernel Version: 6.10.5-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 24 × AMD Ryzen 9 7900X 12-Core Processor
Memory: 62.4 GiB of RAM
Graphics Processor: AMD Radeon RX 7900 GRE

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2024-08-19 20:51:29 UTC 
Where can I install JetBrains Toolbox from? I don't see it available in Discover. Are there any other apps that this happens with? Can you test?
Comment 2 Konrad Materka 2024-08-19 23:39:55 UTC 
It looks like regression. I can see icon in latest Plasma 6.1.4 (Neon), but not in version built from sources. I'll try to build older version tomorrow.

JetBrains Toolbox app icon implementation is strange, but it provides IconPixmap property (with valid pixmap) and IconName:

$ qdbus com.jetbrains.toolbox /StatusNotifierItem org.freedesktop.DBus.Properties.GetAll string:""
Category: ApplicationStatus
IconName: toolbox-tray-color
IconPixmap: qdbus: I don't know how to display an argument of type 'a(iiay)', run with --literal.
IconThemePath: /tmp/.mount_jetbraSGj342
Id: toolbox
ItemIsMenu: false
Menu: /MenuBar
Status: Active
Title: JetBrains Toolbox
Tooltip: JetBrains Toolbox
WindowId: 0
Comment 3 Konrad Materka 2024-08-20 12:34:18 UTC 
I've run bisect, this regressed after https://invent.kde.org/frameworks/kirigami/-/merge_requests/1603
Comment 4 Konrad Materka 2024-08-22 14:16:44 UTC 
This affects also Skype, probably most Electron-based apps - they all use IconThemePath
Comment 5 Bug Janitor Service 2024-08-22 20:27:52 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kirigami/-/merge_requests/1610
Comment 6 Parag W 2024-08-22 20:58:47 UTC 
(In reply to Bug Janitor Service from comment #5)
> A possibly relevant merge request was started @
> https://invent.kde.org/frameworks/kirigami/-/merge_requests/1610

Tested this and it fixes the issue for me.
Comment 7 Nicolas Fella 2024-08-22 22:13:06 UTC 
Git commit 5cfb3bb97213e7e5b4234143ceb5be4865be37a9 by Nicolas Fella.
Committed on 22/08/2024 at 20:27.
Pushed by cullmann into branch 'master'.

[icon] Only reload icon from theme if the theme has that icon

Otherwise we try to reload icons from absolute paths or non-standard search paths, which won't work
Related: bug 491806, bug 491848

M  +1    -1    src/primitives/icon.cpp

https://invent.kde.org/frameworks/kirigami/-/commit/5cfb3bb97213e7e5b4234143ceb5be4865be37a9