Skip to content

native/linux_x11: Allow setting WM_CLASS#532

Merged
not-fl3 merged 1 commit intonot-fl3:masterfrom
bolphen:x11-wm-class
Feb 19, 2025
Merged

native/linux_x11: Allow setting WM_CLASS#532
not-fl3 merged 1 commit intonot-fl3:masterfrom
bolphen:x11-wm-class

Conversation

@bolphen
Copy link
Contributor

@bolphen bolphen commented Feb 19, 2025

Following #501

Since Platform is Copy, I used &'static str instead of String for the name. Shouldn't be a problem cuz you won't want to change it.

Also fixed a typo in icon :)

@not-fl3
Copy link
Owner

not-fl3 commented Feb 19, 2025

Thanks for PR!
Interesting that this is the requirement for gnome, but if it works - it works, I guess!

@not-fl3 not-fl3 merged commit 949b188 into not-fl3:master Feb 19, 2025
11 checks passed
@bolphen
Copy link
Contributor Author

bolphen commented Feb 19, 2025

Yeah it's unfortunate that it doesn't use the X11 icon as a fallback

@bolphen bolphen deleted the x11-wm-class branch February 19, 2025 19:14
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