Botframework-emulator: [MAS4.1.2] [Screen reader-Restart conversation menu] Screen reader is not reading the Name and state for Restart conversation menu

[Kumar2608]

Actual Result:
Screen reader is not reading the Name and state for Restart conversation menu

Expected Result:
Screen reader should read the Name and state for Restart conversation menu
Example: Screen reader should narrate as Restart conversation menu expanded\Collapsed

Note: Issue is repro’ing with NVDA. Please refer attachment below

Repro Steps:

1. Open the Bot Framework V4 Emulator.
2. Welcome tab screen gets opened
   3.Navigate to the open bot and select it
3. Dialog gest opened and connect the open bot
4. Live chat screen gets opened
   6.Navigate to the Restart conversation menu
   7.Observe the issue.

User Impact:
Users who rely on screen reader will be affected to navigate the control if screen reader is not reading the Name and state for menu control screen reader user will be affected.

Test Environment:
OS Build: Version 1903(OS Build 18362.239)
Browser: Edge
Application: Bot Framework V4 Emulator
Screen reader: Narrator

MAS Reference Link:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={248054a6-5e68-4771-9e1e-242fb5025730}
MAS4.1.2_Restart Menu.zip

MAS4.1.2_Restart menu_NVDA.zip

[Kumar2608]

Verified in latest Bot emulator version -4.5.2-79718 .Issue partially fixed. hence we are reactivating this bug
Now NVDA is reading as Restart conversation sub menu 2 of 3 but state is not announcing.

Expected: NVDA should read as "Restart conversation sub menu 2 of 3 expanded\Collapsed .

[corinagum]

@Kumar2608 or @awalia13, could you provide feedback on how best to resolve this issue? The code has aria-expanded applied to both states on this widget. As far as I know we don't have a way of forcing the screen reader to read that state out. Please see below for code showing aria-expanded true and false in both states.

[awalia13]

Had a call with @corinagum. Please find below details.
Code seems good to me. NVDA support aria-haspopup attribute and take precedence over aria-expanded when both attributes are getting used. Because of aria-haspopup attribute NVDA is reading it as Submenu and when control goes to menuitems at that time it is providing full information of menu items along with position which is fine. Hence closing this bug.