Ensure all the screen elements in this Window have their …

xamarin / Xamarin.PropertyEditing

MIT License

24 stars 16 forks source link

Ensure all the screen elements in this Window have their … #724

Closed CartBlanche closed 4 years ago

CartBlanche commented 4 years ago

AutomationProperties.Name property set, for accessibility.

Fixes : https://dev.azure.com/devdiv/DevDiv/_sprints/taskboard/Xamarin%20UI%20Tools/DevDiv/OneVS/CY20-H1/Q2/Sprint%20172?workitem=1004049

This has highlighted the need to run the Accessibility Insights for Windows tool over more of proppy, while it is running in VS. We will also need to do a pass to remove control type or similar.

CartBlanche commented 4 years ago

@decriptor @garuma I'll need a re-review as I had broken a unit test that was looking for a null category. So need to update that to get a green light.

decriptor commented 4 years ago

@CartBlanche The resx file is reformatted again :(

CartBlanche commented 4 years ago

@decriptor unfortunately so. It literally is just the last 4 elements that were added. Both VS4M and VS4W each do different formatting :(