Bug 387874 - Window icon and task manager icon mismatch
Summary: Window icon and task manager icon mismatch
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Task Manager and Icons-Only Task Manager (show other bugs)
Version: 5.8.7
Platform: Debian testing Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-13 17:47 UTC by m
Modified: 2017-12-13 19:19 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot showing icon mismatch (94.09 KB, image/png)
2017-12-13 17:47 UTC, m
Details
.desktop file stored on Desktop (342 bytes, text/plain)
2017-12-13 18:53 UTC, m
Details
Icon png in 128x128 dimensions (18.96 KB, image/png)
2017-12-13 18:55 UTC, m
Details
Screencast of the right icon used briefly in the beginning (627.33 KB, video/mp4)
2017-12-13 18:59 UTC, m
Details

Note You need to log in before you can comment on or make changes to this bug.
Description m 2017-12-13 17:47:08 UTC
Created attachment 109356 [details]
Screenshot showing icon mismatch

Prerequisite:
 - Have Google Chrome installed.

Steps:
1. Open some web page on Google Chrome. Say, https://www.google.com/
2. From the Chrome menu, select Tools > Add to desktop...
3. Make sure 'Open as window' option is checked. Click on Add button.
4. Now a new icon is added to the desktop. Click on it to open the web page in its own window (without any Chrome UI).

Expected:
The icon shown on the window and the one shown on the task bar are the same.

Actual:
Windows shows the web page icon, but task bar shows Chrome icon.
Comment 1 Nate Graham 2017-12-13 18:01:39 UTC
This is due to the differing ways that way KWin and plasmashell work. KWin--which renders the window icon--asks the window for the icon to use, and in this case the window returns the web page icon. The task manager gets the icon from the application's desktop file, in this case the Chrome icon. This is basically the same issue (and same ultimate resolution) as Bug 351055.

At some point this should be resolved in Wayland, with KWin always using the icon from the desktop file.

*** This bug has been marked as a duplicate of bug 351055 ***
Comment 2 m 2017-12-13 18:23:59 UTC
> The task manager gets the icon from the application's desktop file,
> in this case the Chrome icon.

This isn't quite true. The .desktop file has a line like this:

  Icon=chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default

On the file system, I can see that this is the correct icon.

➤ ls ~/.local/share/icons/hicolor/*/apps/chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default*
/usr/local/google/home/mkannan/.local/share/icons/hicolor/128x128/apps/chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default.png
/usr/local/google/home/mkannan/.local/share/icons/hicolor/16x16/apps/chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default.png
/usr/local/google/home/mkannan/.local/share/icons/hicolor/256x256/apps/chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default.png
/usr/local/google/home/mkannan/.local/share/icons/hicolor/32x32/apps/chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default.png
/usr/local/google/home/mkannan/.local/share/icons/hicolor/48x48/apps/chrome-hjpceadhnpnpdelkidbjdmoodafopfkp-Default.png



Also, I failed to mention this in the initial report. For a second or two, the task manager shows the correct icon, and later it changes to Chrome icon.
Comment 3 Nate Graham 2017-12-13 18:28:14 UTC
Interesting. Can you attach the full desktop file?
Comment 4 m 2017-12-13 18:53:58 UTC
Created attachment 109357 [details]
.desktop file stored on Desktop
Comment 5 m 2017-12-13 18:55:02 UTC
Created attachment 109358 [details]
Icon png in 128x128 dimensions
Comment 6 m 2017-12-13 18:59:42 UTC
Created attachment 109359 [details]
Screencast of the right icon used briefly in the beginning

Attached is a screencast video of the task manager using the right icon for a second or two before using the Chrome browser's icon.
Comment 7 Kai Uwe Broulik 2017-12-13 19:04:01 UTC
You're on Plasma 5.8? Try with a more recent version, starting in 5.9 it should treat Chrome apps as separate applications in Task Manager.
Comment 8 m 2017-12-13 19:08:55 UTC
This is a computer supplied by my employer. I cannot (and don't want to) mess with the software configuration. I guess I'll just have to wait till Debian Testing catches up to include 5.9.
Comment 9 Kai Uwe Broulik 2017-12-13 19:14:48 UTC
The patch was too invasive to be included in 5.8 LTS, sorry.
Comment 10 Nate Graham 2017-12-13 19:16:37 UTC
FWIW Debian testing currently has 5.10: https://packages.debian.org/buster/plasma-desktop. Perhaps your company's sysadmin has held it back for some reason.
Comment 11 m 2017-12-13 19:18:36 UTC
@Kai: I totally understand that. It's a relief that the bug is already fixed in v5.9.

@Nate: I see. I'll ask them what's up.


Such quick bug fixes is one of the reasons I have been a KDE user for more than 10 years now. THANK YOU for keeping KDE so great.
Comment 12 Nate Graham 2017-12-13 19:19:30 UTC
:D