Skip to content

fix: #19313 - p-menu Keyboard Navigation broken#19322

Merged
cetincakiroglu merged 1 commit into
masterfrom
issue-#19313
Jan 28, 2026
Merged

fix: #19313 - p-menu Keyboard Navigation broken#19322
cetincakiroglu merged 1 commit into
masterfrom
issue-#19313

Conversation

@mehmetcetin01140

Copy link
Copy Markdown
Contributor

fix: #19313

@github-actions

Copy link
Copy Markdown

Thank you for your contribution! We will review your PR shortly. Please make sure to manually link it to an issue for proper tracking.

@github-actions

Copy link
Copy Markdown

AI Code Review

Looks Good

Summary

Fixes keyboard navigation in p-menu by correcting data-pc-section selector from 'menuitem' to 'item'

Issue Alignment

Directly addresses the issue. The problem was caused by a mismatch between the data-pc-section attribute in the template and the selectors used in keyboard navigation methods. The fix aligns both to use 'item' instead of 'menuitem'.


This is an automated review. A maintainer will provide final approval.

@vercel

vercel Bot commented Jan 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
primeng-5lp7 Ready Ready Preview, Comment Jan 26, 2026 6:24am

@github-actions

Copy link
Copy Markdown

Thanks a lot for your contribution! But, Unit tests failed. You can check the unit tests with the command 'pnpm run test:unit' and commit the changes.

@cetincakiroglu cetincakiroglu merged commit 82aef64 into master Jan 28, 2026
4 of 6 checks passed
@cetincakiroglu cetincakiroglu deleted the issue-#19313 branch January 28, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

p-menu Keyboard Navigation broken

2 participants