Skip to content

Changing StartupWMClass#3701

Open
Kyuyrii wants to merge 1 commit into
mgba-emu:masterfrom
Kyuyrii:patch-1
Open

Changing StartupWMClass#3701
Kyuyrii wants to merge 1 commit into
mgba-emu:masterfrom
Kyuyrii:patch-1

Conversation

@Kyuyrii

@Kyuyrii Kyuyrii commented Feb 9, 2026

Copy link
Copy Markdown

Yo! I'm the publisher of the Snap version, specifically the [Kyu version].

This change is just to avoid this problem here:

Captura de imagem_20260209_184406

@endrift

endrift commented Feb 9, 2026

Copy link
Copy Markdown
Member

@hebo6 you originally added that line. Does this change look correct to you?

@Kyuyrii

Kyuyrii commented Feb 9, 2026

Copy link
Copy Markdown
Author

Just to add some information I discovered, WMClass in Xorg is different from WMClass in Wayland.

This is probably why certain problems like this occur, since many .desktop files may have been created before the migration of the various desktop environments to Wayland.

@endrift

endrift commented Feb 9, 2026

Copy link
Copy Markdown
Member

Yes, I suspected that might be the case. It might be worth trying to set the application name in the QCoreApplication to projectName, which will match the desktop file. See if that works instead.

@endrift

endrift commented Mar 9, 2026

Copy link
Copy Markdown
Member

Poke. I added the change I mentioned in my last post. Can you see if it works without this PR?

@Kyuyrii

Kyuyrii commented Mar 9, 2026

Copy link
Copy Markdown
Author

I'm currently using Gnome, so I can't test it right now, but I'll test it as soon as possible.

@Kyuyrii

Kyuyrii commented Mar 11, 2026

Copy link
Copy Markdown
Author

I built mGBA today, but the problem persists; the mGBA window is recognized as io.mgba.mGBA.

StartupWMClass=io.mgba.mGBA is needed to resolve the issue.

@endrift endrift added this to the mGBA 0.11.0 milestone Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants