Wpf: Narrator ignores IsCheckable on RibbonMenuButtonItem

[ChrisSires]

Expected:
If you set IsCheckable=false, the menu item should not be read in Narrator as checked or un-checked.
Actual:
No matter how IsCheckable is set, Narrator interprets RibbonMenuButtonItems as checkable controls and tries to read them accordingly.
Repro:
Here’s a xaml sample that will exhibit the issue in Narrator:

<Window x:Class="RibbonBug.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    xmlns:local="clr-namespace:RibbonBug"
    mc:Ignorable="d"
    Title="MainWindow" Height="450" Width="800">
    <Grid>
      <Ribbon Title="Ribbon Title">
        <RibbonMenuButton Label="RibbonMenuButton Label">
          <!-- bug: IsCheckable has no effect on Narrator's output the item is still checked or unchecked -->
          <RibbonMenuItem IsCheckable="False" Header="RibbonMenuButtonItem Header" />
        </RibbonMenuButton>
      </Ribbon>
    </Grid>
</Window>

[wjk]

@ChrisSires Try enclosing the XAML in three backticks, like the sample below. You only used one, which won't work.

```xml
<PropertyGroup>
</PropertyGroup>
```

This will make the XAML visible to everyone (as I can't edit your post for you, as I'm not a maintainer of this repo), plus it will be syntax-colored for you (because the "xml" after the first pair of backticks tells GitHub what language it is).

[ChrisSires]

Thanks for the info, fixed!

[stevenbrix]

@ChrisSires, does this also happen with wpf in .net fx?

[ChrisSires]

Setting false appears to have no effect.

From: Vatsan Madhavan [e-mail]
Sent: Thursday, April 4, 2019 7:47 AM
To: dotnet/wpf [e-mail]
Cc: Chris Sires [e-mail]; Mention [e-mail]
Subject: Re: [dotnet/wpf] Narrator ignores IsCheckable on RibbonMenuButtonItem (#349)

Try setting false to prevent clashing with

[vatsan-madhavan]

@arpitmathur , would you please investigate where this issue fits against MAS guidelines?

/cc @grubioe

[arpitmathur]

We're not meeting this particular MAS requirement : MAS 1.3.1 – Info and Relationships. We should consider this for 3.0.

@stevenbrix This issue also reproduces on .net framework.

[arpitmathur]

This has been fixed in PR 2067 in wpf-int.