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

[X11] Allow setting WM_CLASS and _NET_WM_WINDOW_TYPE #14619

Merged
merged 6 commits into from
Feb 29, 2024

Conversation

kekekeks
Copy link
Member

Allows setting X11 specific window properties in the same way we are dealing with Win32 ones.

Open questions: why do we have those in Avalonia.Controls.Platform namespace? It's not XAML-visible by default

@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.1.999-cibuild0044992-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@kekekeks kekekeks marked this pull request as ready for review February 19, 2024 09:38
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.1.999-cibuild0045085-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

Desktop
}

public interface IX11SpecificOptionsToplevelImplFeature
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked in ILSpy, and this interface is fully public there in the ref assembly. I suppose our patch tool only looks for "Impl" in the end of the type name.
image


namespace Avalonia.Controls;

public class X11Properties
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: docs

@kekekeks kekekeks enabled auto-merge February 29, 2024 16:54
Co-authored-by: Max Katz <[email protected]>
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.1.999-cibuild0045447-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@kekekeks kekekeks added this pull request to the merge queue Feb 29, 2024
Merged via the queue into master with commit 8adb67a Feb 29, 2024
7 checks passed
@kekekeks kekekeks deleted the feature/x11-window-type branch February 29, 2024 21:44
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.

3 participants