Skip to content

Commit 2ead12d

Browse files
authored
fix(search): Dynamically Set Maximum Cursor Pointable Item (#85)
### Description <!-- Please add PR description (don't leave blank) - example: This PR [adds/removes/fixes/replaces] the [feature/bug/etc] --> This PR fixes the limitation on the ArrowUp and ArrowDown keybind functionality on the Search Dialog Results. ### Related Issue <!-- Please prefix the issue number with Fixes/Resolves - example: Fixes #123 or Resolves #123 --> Fixes #78 ### Screenshots/Screencasts <!-- Please provide screenshots or video recording that demos your changes (especially if it's a visual change) --> ### Notes to Reviewer <!-- Please state here if you added a new npm packages, or any extra information that can help reviewer better review you changes -->
1 parent ec20d9c commit 2ead12d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/components/islands/search.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,11 +142,11 @@ function SearchDialog() {
142142
const resultsCount = searchResult?.length || 0;
143143
if (resultsCount && e.key === "ArrowUp") {
144144
e.preventDefault();
145-
setCursor(cursor === 0 ? Math.min(resultsCount - 1, 9) : cursor - 1);
145+
setCursor(cursor === 0 ? Math.min(resultsCount - 1, resultsCount) : cursor - 1);
146146
}
147147
if (resultsCount && e.key === "ArrowDown") {
148148
e.preventDefault();
149-
setCursor(cursor === Math.min(resultsCount - 1, 9) ? 0 : cursor + 1);
149+
setCursor(cursor === Math.min(resultsCount - 1, resultsCount) ? 0 : cursor + 1);
150150
}
151151
if (resultsCount && e.key === "Enter") {
152152
e.preventDefault();

0 commit comments

Comments
 (0)