KEMBAR78
[Windows] Revert cleanup of `AccessibilityExtensions` by MartyIX · Pull Request #31608 · dotnet/maui · GitHub
Skip to content

Conversation

MartyIX
Copy link
Contributor

@MartyIX MartyIX commented Sep 14, 2025

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description of Change

Partial revert of #27052. This PR adds a test to make sure that any new PR does not introduce a similar regression.

Issues Fixed

Fixes #31606

@dotnet-policy-service
Copy link
Contributor

Hey there @@MartyIX! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@jfversluis
Copy link
Member

/azp run MAUI-public

@jfversluis jfversluis added this to the .NET 9 SR12 milestone Sep 15, 2025
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jfversluis jfversluis added the p/0 Work that we can't release without label Sep 15, 2025
@PureWeen PureWeen moved this from Todo to Ready To Review in MAUI SDK Ongoing Sep 16, 2025
Copy link
Member

@PureWeen PureWeen left a comment

Choose a reason for hiding this comment

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

@MartyIX should this have a UITest part?

@github-project-automation github-project-automation bot moved this from Ready To Review to Changes Requested in MAUI SDK Ongoing Sep 16, 2025
@MartyIX
Copy link
Contributor Author

MartyIX commented Sep 16, 2025

@MartyIX should this have a UITest part?

I added it intentionally to make sure that it works for customers (it's a bit surprising that there is no test that would catch it). The PR can be a simple revert and it would be OK as well.

@PureWeen PureWeen moved this from Changes Requested to Ready To Review in MAUI SDK Ongoing Sep 16, 2025
jfversluis
jfversluis previously approved these changes Sep 16, 2025
@jfversluis
Copy link
Member

jfversluis commented Sep 16, 2025

/azp run MAUI-UITests-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@PureWeen PureWeen left a comment

Choose a reason for hiding this comment

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

Still needs host code to run the test

Merge when that's added

@github-project-automation github-project-automation bot moved this from Ready To Review to Changes Requested in MAUI SDK Ongoing Sep 16, 2025
@MartyIX
Copy link
Contributor Author

MartyIX commented Sep 16, 2025

Added the test code. Please AZP for me to upload snapshots later.

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PureWeen
Copy link
Member

/azp run MAUI-UITests-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MartyIX MartyIX force-pushed the feature/2025-09-14-revert-27052 branch from 8b2496e to 5742cf4 Compare September 16, 2025 13:56
@PureWeen
Copy link
Member

/azp run MAUI-UITests-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PureWeen PureWeen moved this from Changes Requested to Ready To Review in MAUI SDK Ongoing Sep 16, 2025
@github-project-automation github-project-automation bot moved this from Ready To Review to Approved in MAUI SDK Ongoing Sep 16, 2025
@PureWeen
Copy link
Member

/azp run MAUI-DeviceTests-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PureWeen PureWeen changed the base branch from main to inflight/candidate September 18, 2025 20:23
@PureWeen PureWeen merged commit 8faf90c into dotnet:inflight/candidate Sep 18, 2025
111 of 113 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in MAUI SDK Ongoing Sep 18, 2025
@MartyIX MartyIX deleted the feature/2025-09-14-revert-27052 branch September 18, 2025 20:25
github-actions bot pushed a commit that referenced this pull request Sep 18, 2025
github-actions bot pushed a commit that referenced this pull request Sep 23, 2025
github-actions bot pushed a commit that referenced this pull request Sep 23, 2025
@PureWeen
Copy link
Member

/backport to release/9.0.1xx-sr11

@github-actions
Copy link
Contributor

Started backporting to release/9.0.1xx-sr11: https://github.com/dotnet/maui/actions/runs/17977802558

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community ✨ Community Contribution p/0 Work that we can't release without platform/windows

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

MAUI 9.0.110 crashes on Windows when SemanticProperties.Description is set for ToolbarItem

4 participants