Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Task Switcher icon is missing on Plasma Wayland #154495

Closed
mamantoha opened this issue Jul 8, 2022 · 12 comments
Closed

Task Switcher icon is missing on Plasma Wayland #154495

mamantoha opened this issue Jul 8, 2022 · 12 comments
Assignees

Comments

@mamantoha
Copy link

mamantoha commented Jul 8, 2022

Does this issue occur when all extensions are disabled?: Yes/No

  • VS Code Version: 1.69.0
  • OS Version: Manjaro Linux 21.3.2
  • DE: Plasma 5.24.5

Steps to Reproduce:

  1. Install vscode from AUR https://aur.archlinux.org/packages/visual-studio-code-bin
  2. Icon in Task Switcher (Alt+Tab) is shown as a fallback Wayland icon

Screenshot_20220708_141653

However, an icon in Task Manager is ok.

Probably this issue is because vscode window missing property "desktopFileName"
Kwin Debug Console:
Screenshot_20220708_141909

But I'm not sure, it can be an issue because of something else.

See https://community.kde.org/Guidelines_and_HOWTOs/Wayland_Porting_Notes#Application_Icon for reference:

If for some reason your application is not using KAboutData, you need to manually call QGuiApplication::setDesktopFileName().

I believe there is something similar for Electron apps.

Related Electron issue: electron/electron#34852

@nocturnalmc
Copy link

Have the same issue on Gnome too where launching it on Wayland causing the icon on the dock to be missing

@edipangi
Copy link

edipangi commented Jul 9, 2022

It is also present in insiders build.

@Merrit
Copy link

Merrit commented Jul 9, 2022

Seems very easy to test:

  • Running code: normal
  • Running code --ozone-platform=wayland: generic icon in taskbar / overview

@sysoev-dev
Copy link

@Merrit
Fedora 36, Gnome (Wayland)
code - starting on Xwayland. Icon present
code --ozone-platform=wayland - Using wayland. Icon missing

@Merrit
Copy link

Merrit commented Jul 13, 2022

@Merrit Fedora 36, Gnome (Wayland) code - starting on Xwayland. Icon present code --ozone-platform=wayland - Using wayland. Icon missing

Yes, exactly.

@sysoev-dev
Copy link

Yes, exactly.

I'm annoyed by the lack of an icon. But, I still choose wayland because it is more productive. On xwayland I get subtle artifacts

@nocturnalmc
Copy link

Yes, exactly.

I'm annoyed by the lack of an icon. But, I still choose wayland because it is more productive. On xwayland I get subtle artifacts

For me on xwayland the cursor is lagging a lot. On wayland the cursor move snappier

@ReillyBrogan
Copy link

I just updated Electron to 19.0.10 locally (by applying a slightly modified #154092), and it appears that this behavior is caused because in Wayland the app_id is set to code-oss-url-handler while in X11 the WMClass is set to Code - OSS. These need to be identical for the window manager to correctly correlate the running applications to the desktop file so that it will show the correct icon in the task manager and any task switcher applications.

@ReillyBrogan
Copy link

For instance if you change the StartupWMClass field (despite the name this is matched to the app_id in Wayland) in the desktop file to code-oss-url-handler and reload the application database then the icon will be shown correctly when VSCode is launched in Wayland but the icon will be broken when launched under X11.

@mamantoha
Copy link
Author

This is already fixed in Electron electron/electron#34877 v18.3.6 and v19.0.9.

@ReillyBrogan
Copy link

This is already fixed in Electron electron/electron#34877 v18.3.6 and v19.0.9.

Who are you replying to? Because if you're replying to me I already stated that my observations came from testing 19.0.10

@deepak1556
Copy link
Contributor

Merging to #154693

@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants