| Summary: | mainwindow icon bug | ||
|---|---|---|---|
| Product: | [Applications] kleopatra | Reporter: | fxzxmic |
| Component: | general | Assignee: | Andre Heinecke <aheinecke> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | fxzxmic, kdepim-bugs, kossebau, mutz |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/pim/kleopatra/commit/9dfe6709cff75f777310968bb30151599e68dd0c | Version Fixed In: | |
| Sentry Crash Report: | |||
| Attachments: | bug related screenshot | ||
|
Description
fxzxmic
2023-01-13 08:43:03 UTC
As commented on irc, I suspect the cause is that the window manager used here is an X11 one which only fetches the window icons from the X properties directly, does not map the _KDE_NET_WM_DESKTOP_FILE to a desktop file to fetch metadata like the app icon.
For those one still needs to set the window icon directly, e.g. by adding
app.setWindowIcon(QIcon::fromTheme(QStringLiteral("kleopatra"), app.windowIcon()));
to https://invent.kde.org/pim/kleopatra/-/blob/master/src/main.cpp#L128
While at it, all the app metadata setting is better moved to before the "// Initialize GpgME" section, otherwise e.g. the text in the warning message will not be translated properly I expect as well as the window again missing needed metadata to be nicely shown as Kleopatra window.
Git commit 9dfe6709cff75f777310968bb30151599e68dd0c by Ingo Klöcker, on behalf of Ingo Klöcker. Committed on 20/01/2023 at 14:14. Pushed by kloecker into branch 'master'. Explicitly set the window icon Apparently, some window managers need the icon to be set explicitly. M +2 -0 src/main.cpp https://invent.kde.org/pim/kleopatra/commit/9dfe6709cff75f777310968bb30151599e68dd0c Which version will this change be included in? |