On Fri, 31 Mar 2023 15:27:50 GMT, Abhishek Kumar <[email protected]> wrote:
> It seems that the accessibility role returned by the MenuAccessibility class > is not correct for top level menu. > > Added a condition check to return the accessibility role as > `NSAccessibilityMenuBarItemRole` in stead of `NSAccessibilityMenuRole` for > top level menu whose parent is `menubar`. > > The solution works correctly for screen magnifier as well as voice over. > > A manual test case is added to verify the fix. > > Note: Tried implementing few a11y methods to fix but for top level menu with > `NSAccessibilityMenuRole` doesn't return the a11y label. This pull request has now been integrated. Changeset: ecec611a Author: Abhishek Kumar <[email protected]> URL: https://git.openjdk.org/jdk/commit/ecec611af6c6314d7a834392f38468ad3f390e2d Stats: 112 lines in 2 files changed: 109 ins; 0 del; 3 mod 8283404: [macos] a11y : Screen magnifier does not show JMenu name Reviewed-by: serb ------------- PR: https://git.openjdk.org/jdk/pull/13271
