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.

Sign up for GitHub

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

Jump to bottom

ListView Default Keyboard Accelerators #8063

Closed
ferrariofilippo opened this issue Jan 4, 2023 · 3 comments
Closed

ListView Default Keyboard Accelerators #8063

ferrariofilippo opened this issue Jan 4, 2023 · 3 comments
Labels
area-Accessibility area-KeyboardAccelerators area-Lists ListView, GridView, ListBox, etc bug Something isn't working team-Controls Issue for the Controls team
Milestone
WinAppSDK 1.5

Comments

@ferrariofilippo
Copy link

Describe the bug

Currently, ListView handles Ctrl + Alt + A (or equivalent) by default, selecting all its items. This is a problem since, if the listview contains editable items (such as TextBoxes), the user can't enter specific characters, such as Ā (Ctrl + Alt + A - Latvian Keyboard Layout).

Steps to reproduce the bug

  1. Press Ctrl + Alt + A on a ListView
  2. See that all items are selected

Expected behavior

When the user presses Ctrl + Alt + A (or AltGr + A), listview should not handle KeyboardAccelerator

Screenshots

No response

NuGet package version

None

Windows version

Windows 11 (22H2): Build 22621

Additional context

No response
@ferrariofilippo ferrariofilippo added the bug Something isn't working label Jan 4, 2023
@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Jan 4, 2023
@ferrariofilippo ferrariofilippo mentioned this issue Jan 4, 2023
Closed
@gabbybilka gabbybilka added area-Lists ListView, GridView, ListBox, etc area-KeyboardAccelerators team-Controls Issue for the Controls team area-Accessibility and removed needs-triage Issue needs to be triaged by the area owners labels Jan 6, 2023
@github-actions
Copy link

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@ferrariofilippo
Copy link
Author

I'll keep this open since it's still an issue

@JJBrychell JJBrychell added the fixed-internally This bug has been fixed, and the fix will be shipped in the next version of WinUI 3. label Jan 26, 2024
@JJBrychell
Copy link

The ListView control was looking to see if the control key was pressed to determine its accelerators, but it wasn't looking to see if any other modifier was pressed as well.

The fix is in and projected for the 1.5 release.

@bpulliam bpulliam added this to the WinAppSDK 1.5 milestone Feb 6, 2024
@bpulliam bpulliam removed the fixed-internally This bug has been fixed, and the fix will be shipped in the next version of WinUI 3. label Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-Accessibility area-KeyboardAccelerators area-Lists ListView, GridView, ListBox, etc bug Something isn't working team-Controls Issue for the Controls team
Projects
None yet
Milestone
WinAppSDK 1.5
Development

No branches or pull requests
4 participants
@JJBrychell @bpulliam @gabbybilka @ferrariofilippo