[Android] Fix a11y set semantic focus (#1782)

rachelkang · TheCodeTraveler · jsuarezruiz · web-flow · commit a4557b74cac8 · 2022-01-11T10:30:00.000+01:00
* Add SemanticExtensions SetFocus and Announce

* Add Windows Announce implementation

* Update based on feedback

* Fix SetSemanticFocus for Android

* Revert order of usings to clean up diff

Co-authored-by: Brandon Minnick &lt;13558917+brminnick@users.noreply.github.com&gt;
Co-authored-by: Javier Suárez &lt;javiersuarezruiz@hotmail.com&gt;
Co-authored-by: Gerald Versluis &lt;gerald.versluis@microsoft.com&gt;
diff --git a/src/CommunityToolkit/Xamarin.CommunityToolkit/Extensions/Semantic/SemanticExtensions.android.cs b/src/CommunityToolkit/Xamarin.CommunityToolkit/Extensions/Semantic/SemanticExtensions.android.cs
@@ -17,7 +17,7 @@ static void PlatformSetSemanticFocus(this VisualElement element)
 			if (androidView is not AView view)
 				throw new NullReferenceException("Can't access view");
 
-			view.SendAccessibilityEvent(EventTypes.ViewFocused);
+			view.SendAccessibilityEvent((EventTypes)(int)WindowsChange.AccessibilityFocused);
 		}
 
 		static void PlatformAnnounce(string text)
@@ -36,4 +36,4 @@ static void PlatformAnnounce(string text)
 			manager.SendAccessibilityEvent(announcement);
 		}
 	}
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ static void PlatformSetSemanticFocus(this VisualElement element)`
`17`	`17`	`if (androidView is not AView view)`
`18`	`18`	`throw new NullReferenceException("Can't access view");`
`19`	`19`
`20`		`- view.SendAccessibilityEvent(EventTypes.ViewFocused);`
	`20`	`+ view.SendAccessibilityEvent((EventTypes)(int)WindowsChange.AccessibilityFocused);`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`static void PlatformAnnounce(string text)`
`@@ -36,4 +36,4 @@ static void PlatformAnnounce(string text)`
`36`	`36`	`manager.SendAccessibilityEvent(announcement);`
`37`	`37`	`}`
`38`	`38`	`}`
`39`		`-}`
	`39`	`+}`