[kwin] [Bug 468129] Display window icon from app's .desktop file if no icon was set by the window

2023-04-03 Thread Wyatt Childers
https://bugs.kde.org/show_bug.cgi?id=468129

--- Comment #7 from Wyatt Childers  ---
(In reply to Nicolas Fella from comment #6)
> Nate, I'm afraid you are mistaken.
> 
> On Wayland there is no concept of a window icon. The icon you see for
> Wayland apps *always* comes from the desktop file. Now why are some icons
> missing? This happens when the desktop file name reported by the app doesn't
> match the actual desktop file on disk. Plasma has some extra code that tries
> to fix up these broken apps that kwin doesn't have, which is why some apps
> have icons in Plasma but not in KWin

Perhaps that logic could be moved into one of the KDE frameworks, and then
shared between Plasma and Kwin so things are always consistent?

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 468129] Display window icon from app's .desktop file if no icon was set by the window

2023-04-03 Thread Nicolas Fella
https://bugs.kde.org/show_bug.cgi?id=468129

Nicolas Fella  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 CC||nicolas.fe...@gmx.de
 Status|CONFIRMED   |REPORTED

--- Comment #6 from Nicolas Fella  ---
Nate, I'm afraid you are mistaken.

On Wayland there is no concept of a window icon. The icon you see for Wayland
apps *always* comes from the desktop file. Now why are some icons missing? This
happens when the desktop file name reported by the app doesn't match the actual
desktop file on disk. Plasma has some extra code that tries to fix up these
broken apps that kwin doesn't have, which is why some apps have icons in Plasma
but not in KWin

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 468129] Display window icon from app's .desktop file if no icon was set by the window

2023-04-03 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=468129

Nate Graham  changed:

   What|Removed |Added

 Status|REOPENED|CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 468129] Display window icon from app's .desktop file if no icon was set by the window

2023-04-03 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=468129

Nate Graham  changed:

   What|Removed |Added

Summary|Flatpak window icons are|Display window icon from
   |wrong   |app's .desktop file if no
   ||icon was set by the window
   Severity|normal  |wishlist
 Resolution|DOWNSTREAM  |---
 Ever confirmed|0   |1
 Status|RESOLVED|REOPENED

--- Comment #5 from Nate Graham  ---
Yes, that seems reasonable to me.

-- 
You are receiving this mail because:
You are watching all bug changes.