Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SettingsCard missing invoke pattern #391

Open
2 of 24 tasks
AmelBawa-msft opened this issue Apr 2, 2024 · 1 comment
Open
2 of 24 tasks

SettingsCard missing invoke pattern #391

AmelBawa-msft opened this issue Apr 2, 2024 · 1 comment
Assignees
Labels
accessibility :accessibility: ux 🖌️ Everything design related

Comments

@AmelBawa-msft
Copy link

Describe the bug

Issue

Accessibility Insights reporting: "A button must support one of these patterns: Invoke, Toggle, or ExpandCollapse." when using a SettingsCard that is clickable (IsClickEnabled=True).

Investigation

  • SettingsCardAutomationPeer may possibly need to implement GetPatternCore(PatternInterface patternInterface) method.
protected override object GetPatternCore(PatternInterface patternInterface)
{
    return patternInterface == PatternInterface.Invoke ? this : base.GetPatternCore(patternInterface);
}
  • Implement IInvokeProvider in the same class (since we're returning this).
  • Or instead, maybe the SettingsCardAutomationPeer can inherit from ButtonBaseAutomationPeer?

Steps to reproduce

- Add a settings card

<controls:SettingsCard Header="Example" IsClickEnabled="True" />


- Run accessibility scan on the displayed card
- The following error is reported: "A button must support one of these patterns: Invoke, Toggle, or ExpandCollapse."

Expected behavior

No errors

Screenshots

No response

Code Platform

  • UWP
  • WinAppSDK / WinUI 3
  • Web Assembly (WASM)
  • Android
  • iOS
  • MacOS
  • Linux / GTK

Windows Build Number

  • Windows 10 1809 (Build 17763)
  • Windows 10 1903 (Build 18362)
  • Windows 10 1909 (Build 18363)
  • Windows 10 2004 (Build 19041)
  • Windows 10 20H2 (Build 19042)
  • Windows 10 21H1 (Build 19043)
  • Windows 10 21H2 (Build 19044)
  • Windows 10 22H2 (Build 19045)
  • Windows 11 21H2 (Build 22000)
  • Other (specify)

Other Windows Build number

No response

App minimum and target SDK version

  • Windows 10, version 1809 (Build 17763)
  • Windows 10, version 1903 (Build 18362)
  • Windows 10, version 1909 (Build 18363)
  • Windows 10, version 2004 (Build 19041)
  • Windows 10, version 2104 (Build 20348)
  • Windows 11, version 22H2 (Build 22000)
  • Other (specify)

Other SDK version

No response

Visual Studio Version

No response

Visual Studio Build Number

No response

Device form factor

No response

Additional context

No response

Help us help you

Yes, I'd like to be assigned to work on this item.

@michael-hawker
Copy link
Member

michael-hawker commented Apr 4, 2024

Thanks @AmelBawa-msft for the report and offering to help. We're working towards an 8.1 release in the near future. This seems like a simple fix, if you can open a PR this week, we can get look to get it into the upcoming release. FYI @niels9001

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility :accessibility: ux 🖌️ Everything design related
Projects
Status: 🏗 In progress
Development

No branches or pull requests

2 participants