@oh-my-pi/pi-tui 16.1.12 → 16.1.14

package/dist/types/components/settings-list.d.ts

@@ -82,6 +82,8 @@ export declare class SettingsList implements Component {
     setMaxVisible(rows: number): void;
     /** Move the selection one step for a wheel notch. */
     handleWheel(delta: -1 | 1): void;
+    /** Move the selection one step for a wheel notch if the pointer is within the settings pane. */
+    handleWheelAt(delta: -1 | 1, _line: number, col: number): boolean;
     /** Highlight the item under the pointer (null clears). */
     setHoverItem(id: string | null): void;
     /**

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-tui",
-	"version": "16.1.12",
+	"version": "16.1.14",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -37,8 +37,8 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "16.1.12",
-		"@oh-my-pi/pi-utils": "16.1.12",
+		"@oh-my-pi/pi-natives": "16.1.14",
+		"@oh-my-pi/pi-utils": "16.1.14",
 		"lru-cache": "11.5.1",
 		"marked": "^18.0.5"
 	},

package/src/components/settings-list.ts

@@ -195,7 +195,17 @@ export class SettingsList implements Component {
 		if (this.#submenuComponent) return;
 		// Wheel is row-level interaction: it returns focus to the rows.
 		this.#sectionFocus = false;
-		this.#moveSelection(delta);
+		this.#moveSelection(delta, false);
+	}
+
+	/** Move the selection one step for a wheel notch if the pointer is within the settings pane. */
+	handleWheelAt(delta: -1 | 1, _line: number, col: number): boolean {
+		if (this.#submenuComponent) return false;
+		if (this.#sidebarHitCol > 0 && col < this.#sidebarHitCol) {
+			return false;
+		}
+		this.handleWheel(delta);
+		return true;
 	}
 
 	/** Highlight the item under the pointer (null clears). */
@@ -308,13 +318,22 @@ export class SettingsList implements Component {
 		return index >= 0 ? index : 0;
 	}
 
-	/** Move selection by one selectable item, wrapping and skipping headings. */
-	#moveSelection(delta: -1 | 1): void {
+	/** Move selection by one selectable item, wrapping or clamping, and skipping headings. */
+	#moveSelection(delta: -1 | 1, wrap = true): void {
 		const len = this.#filteredItems.length;
 		if (len === 0) return;
 		let index = this.#selectedIndex;
-		for (let step = 0; step < len; step++) {
-			index = (index + delta + len) % len;
+		for (let step = 0; step < len * 2; step++) {
+			const next = index + delta;
+			if (next < 0 || next >= len) {
+				if (wrap) {
+					index = (next + len) % len;
+				} else {
+					return;
+				}
+			} else {
+				index = next;
+			}
 			if (!this.#filteredItems[index]?.heading) {
 				this.#selectedIndex = index;
 				this.#notifySelection();