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

Fix context menu keyboard selection. #16354

Merged
merged 3 commits into from
Jul 20, 2024
Merged

Conversation

grokys
Copy link
Member

@grokys grokys commented Jul 17, 2024

What does the pull request do?

When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by #11287. The fix is to make the ContextMenu focusable.

Added an integration test to try to prevent regressions.

When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by #11287. The fix is to make the `ContextMenu` focusable.

Added an integration test to try to prevent regressions.
@grokys grokys added bug customer-priority Issue reported by a customer with a support agreement. backport-candidate-11.1.x Consider this PR for backporting to 11.1 branch labels Jul 17, 2024
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0050238-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@grokys grokys enabled auto-merge July 18, 2024 08:38
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0050285-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@grokys grokys added this pull request to the merge queue Jul 20, 2024
Merged via the queue into master with commit 0e86973 Jul 20, 2024
11 checks passed
@grokys grokys deleted the fixes/context-menu-keyboard branch July 20, 2024 22:32
grokys added a commit that referenced this pull request Jul 23, 2024
When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by #11287. The fix is to make the `ContextMenu` focusable.

Added an integration test to try to prevent regressions.

Co-authored-by: Max Katz <[email protected]>
Gillibald pushed a commit to Gillibald/Avalonia that referenced this pull request Jul 29, 2024
When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by AvaloniaUI#11287. The fix is to make the `ContextMenu` focusable.

Added an integration test to try to prevent regressions.

Co-authored-by: Max Katz <[email protected]>
Gillibald pushed a commit that referenced this pull request Jul 30, 2024
When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by #11287. The fix is to make the `ContextMenu` focusable.

Added an integration test to try to prevent regressions.

Co-authored-by: Max Katz <[email protected]>
grokys added a commit that referenced this pull request Jul 31, 2024
When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by #11287. The fix is to make the `ContextMenu` focusable.

Added an integration test to try to prevent regressions.

Co-authored-by: Max Katz <[email protected]>
@grokys grokys added backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch backported-11.0.x and removed backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch labels Jul 31, 2024
grokys added a commit that referenced this pull request Aug 2, 2024
When opening a context menu and pressing the "arrow down" key, the first menu item was not selected. This issue was introduced by #11287. The fix is to make the `ContextMenu` focusable.

Added an integration test to try to prevent regressions.

Co-authored-by: Max Katz <[email protected]>
@grokys grokys added backported-11.1.x and removed backport-candidate-11.1.x Consider this PR for backporting to 11.1 branch labels Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-11.0.x backported-11.1.x bug customer-priority Issue reported by a customer with a support agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants