From 51e792ccdf37aa1bcf24b00ade2b58e53a45a454 Mon Sep 17 00:00:00 2001
From: kijai <40791699+kijai@users.noreply.github.com>
Date: Sat, 15 Mar 2025 11:27:21 +0200
Subject: [PATCH] Allow setting Set/Get node position in node suggestion menu

---
 web/js/contextmenu.js | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/web/js/contextmenu.js b/web/js/contextmenu.js
index 3b1670e..200fb38 100644
--- a/web/js/contextmenu.js
+++ b/web/js/contextmenu.js
@@ -49,7 +49,7 @@ app.registerExtension({
 	},
 		async setup(app) {
 			const onChange = (value) => {
-				if (value) {
+				if (value!="disabled") {
 					const valuesToAddToIn = ["GetNode"];
 					const valuesToAddToOut = ["SetNode"];
 			
@@ -59,7 +59,11 @@ app.registerExtension({
 							if (idx !== -1) {
 								arr.splice(idx, 1);
 							}
-							arr.unshift(valueToAdd);
+							if (value === "top") {
+								arr.unshift(valueToAdd);
+							} else {
+								arr.push(valueToAdd);
+							}
 						}
 					}
 			
@@ -69,7 +73,11 @@ app.registerExtension({
 							if (idx !== -1) {
 								arr.splice(idx, 1);
 							}
-							arr.unshift(valueToAdd);
+							if (value === "top") {
+								arr.unshift(valueToAdd);
+							} else {
+								arr.push(valueToAdd);
+							}
 						}
 					}
 				}
@@ -77,9 +85,11 @@ app.registerExtension({
 			
 			app.ui.settings.addSetting({
 				id: "KJNodes.SetGetMenu",
-				name: "KJNodes: Make Set/Get -nodes defaults (turn off and reload to disable)",
-				defaultValue: false,
-				type: "boolean",
+				name: "KJNodes: Make Set/Get -nodes defaults",
+				tooltip: 'Adds Set/Get nodes to the top or bottom of the list of available node suggestions. Disabling requires a browser reload.',
+				options: ['disabled', 'top', 'bottom'],
+				defaultValue: 'disabled',
+				type: "combo",
 				onChange: onChange,
 				
 			});