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

native/linux_x11: Allow setting WM_CLASS #532

Merged
merged 1 commit into from
Feb 19, 2025

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