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

Add basic integration tests for NativeControlHost and improve its automation/a11y support. #15542

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

grokys
Copy link
Member

@grokys grokys commented Apr 27, 2024

What does the pull request do?

Adds a couple of basic integration tests for NativeControlHost. To do this I had to improve automation/a11y support for embedding native controls. More description to come...

grokys and others added 7 commits April 26, 2024 12:19
Currently embeds a lone native text box, and only when running on Windows.
Allows native controls to appear in the Avalonia automation tree as a child of the `NativeControlHost`. They also appear in the _wrong_ place - as a direct child of the `Window` - but this appears to be expected behavior as it happens when hosting a win32 control in WPF as well.
Implements special-casing of `InteropAutomationPeer`on macOS.
The test for the native control in a popup is disabled on macOS because we have a bug there.
@avaloniaui-bot
Copy link

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

@cla-avalonia
Copy link
Collaborator

cla-avalonia commented Apr 27, 2024

  • All contributors have signed the CLA.

@grokys
Copy link
Member Author

grokys commented Apr 28, 2024

@cla-avalonia agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants