Skip to content

Conversation

@manthan2325
Copy link

Checklist

@manthan2325 manthan2325 requested a review from a team as a code owner December 27, 2025 18:53
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Dec 27, 2025
@manthan2325
Copy link
Author

HI @weeman1337 could you please add the T-enhancement label to my PR so the checks can pass ? Thanks !

Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must describe your change. Your title offers no insight and you have no PR description.

const onDialogKeyDown = (ev: KeyboardEvent | React.KeyboardEvent): void => {
const key = (ev as KeyboardEvent).key;
// handle PageUp / PageDown to scroll the results container by one page
if (key === "PageDown" || key === "PageUp") {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be using the keybinding stack as below

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants