[shgraph] Review fixes

bin/style.css

@@ -4500,7 +4500,6 @@ fancy-array fancy-items fancy-item fancy-item-content {
   padding: 0.2em 1em;
   padding-right: 0px;
   padding-right: 0;
-  /* border-left: 1px solid var(--border-color); */
   margin-bottom: 0.5em;
 }
 fancy-array fancy-items fancy-item .toggle-open * {

bin/style.less

@@ -5280,109 +5280,11 @@ fancy-separator {
 	height: 0;
 	min-width: 0;
 	max-width: 0;
-	// border-left: 1px solid #444;
 	padding: 0;
 	margin: 0 0.5em;
 	height: 16px;
 }
 
-// fancy-array {
-// 	* {
-// 		box-sizing: border-box;
-// 	}
-
-// 	overflow-y: auto;
-// 	overflow-x: clip;
-
-// 	&::-webkit-scrollbar {
-// 		width: 3px;
-
-// 		background: rgba(127,127,127,0.5);
-// 	}
-
-// 	&::-webkit-scrollbar-thumb {
-// 		background: #AAA;
-// 		border-radius: 5px;
-// 	}
-
-// 	border: 1px solid #333;
-// 	border-radius: 5px;
-
-
-// 	display: flex;
-// 	flex-direction: column;
-
-// 	border-radius: 5px;
-// 	padding: 2em;
-
-// 	gap: 2px;
-
-// 	width: 100%;
-
-// 	justify-content: stretch;
-
-// 	fancy-list-item {
-// 		display: flex;
-// 		flex-direction: column;
-
-// 		fancy-list-header {
-// 			display: flex;
-
-// 			color: var(--quiet-text-color);
-// 			border-radius: 3px;
-
-// 			background-color: #444;
-
-// 			>input[type="text"] {
-// 				min-width: 0;
-// 				width: 0;
-
-// 				flex-grow: 1;
-
-// 				background: none;
-// 				outline: none;
-// 				border: none;
-
-// 				color: var(--main-text-color);
-// 				font-size: 14px;
-
-// 				&:hover {
-// 					outline: 1px solid var(--border-color);
-// 					border-radius: 3px;
-// 				}
-// 			}
-// 		}
-
-// 		fancy-list-content {
-// 			display: none;
-// 			margin-top: 2px;
-
-// 			padding:  0.2em 1em;
-// 			padding-right: 0;
-// 			margin-right: calc(-0.2em - 1px);
-
-
-// 			/* border-left: 1px solid var(--border-color); */
-
-// 			margin-bottom: 0.5em;
-// 		}
-
-// 		.open-button .ico {
-// 			transition: transform 0.2s;
-// 		}
-
-// 		&.open {
-// 			> fancy-list-header .open-button .ico {
-// 				transform: rotate(90deg);
-// 			}
-
-// 			> fancy-list-content{
-// 				display: block;
-// 			}
-// 		}
-// 	}
-// }
-
 fancy-array {
 	box-sizing: border-box;
 
@@ -5459,9 +5361,6 @@ fancy-array {
 				padding-right: 0px;
 				padding-right: 0;
 
-
-				/* border-left: 1px solid var(--border-color); */
-
 				margin-bottom: 0.5em;
 			}
 
@@ -5479,7 +5378,6 @@ fancy-array {
 				}
 			}
 
-
 			&.hovertop:before, &.hoverbot:after {
 				display: block;
 				position: absolute;

hide/view/animgraph/AnimGraphEditor.hx

@@ -367,132 +367,6 @@ class AnimGraphEditor extends GenericGraphEditor {
 
     function refreshPamamList() {
         parametersList.refresh();
-        // for (paramIndex => param in animGraph.parameters) {
-        //     var paramElement = new Element('<graph-parameter>
-        //         <header>
-        //             <div class="reorder ico ico-reorder" draggable="true"></div>
-        //             <div class="ico ico-chevron-down toggle-open"></div>
-        //             <input type="text" value="${param.name}" class="fill"></input>
-        //             <button-2 class="menu"><div class="ico ico-ellipsis-v"/></button-2>
-        //         </header>
-        //     </graph-parameter>').appendTo(parametersList);
-
-        //     var open : Bool = getDisplayState('param.${paramIndex}') ?? false;
-        //     paramElement.toggleClass("folded", open);
-
-        //     var name = paramElement.find("input");
-        //     name.on("change", (e) -> {
-        //         var prev = param.name;
-        //         var curr = name.val();
-
-        //         function exec(isUndo: Bool) {
-        //             if (!isUndo) {
-        //                 param.name = curr;
-        //             } else {
-        //                 param.name = prev;
-        //             }
-        //             name.val(param.name);
-        //             var toRefresh = animGraph.nodes.filter((n) -> Std.downcast(n, hrt.animgraph.nodes.FloatParameter)?.parameter == param);
-        //             for (node in toRefresh) {
-        //                 graphEditor.refreshBox(node.id);
-        //             }
-        //         }
-
-        //         exec(false);
-        //         undo.change(Custom(exec));
-        //     });
-
-        //     name.on("contextmenu", (e) -> {
-        //         e.stopPropagation();
-        //     });
-
-        //     var toggleOpen = paramElement.find(".toggle-open");
-        //     toggleOpen.on("click", (e) -> {
-        //         open = !open;
-        //         saveDisplayState('param.${paramIndex}', open);
-        //         paramElement.toggleClass("folded", open);
-        //     });
-
-        //     var reorder = paramElement.find(".reorder");
-        //     reorder.get(0).ondragstart = (e: js.html.DragEvent) -> {
-        //         e.dataTransfer.setDragImage(paramElement.get(0), Std.int(paramElement.width()), 0);
-
-        //         e.dataTransfer.setData(parametersList.getDragKeyName(), '${paramIndex}');
-        //         e.dataTransfer.dropEffect = "move";
-        //     }
-
-        //     inline function isAfter(e) {
-        //         return e.clientY > (paramElement.offset().top + paramElement.outerHeight() / 2.0);
-        //     }
-
-        //     paramElement.get(0).addEventListener("dragover", function(e : js.html.DragEvent) {
-        //         if (!e.dataTransfer.types.contains(parametersList.getDragKeyName()))
-        //             return;
-        //         var after = isAfter(e);
-        //         paramElement.toggleClass("hovertop", !after);
-        //         paramElement.toggleClass("hoverbot", after);
-        //         e.preventDefault();
-        //     });
-
-        //     paramElement.get(0).addEventListener("dragleave", function(e : js.html.DragEvent) {
-        //         if (!e.dataTransfer.types.contains(parametersList.getDragKeyName()))
-        //             return;
-        //         paramElement.toggleClass("hovertop", false);
-        //         paramElement.toggleClass("hoverbot", false);
-        //     });
-
-        //     paramElement.get(0).addEventListener("dragenter", function(e : js.html.DragEvent) {
-        //         if (!e.dataTransfer.types.contains(parametersList.getDragKeyName()))
-        //             return;
-        //         e.preventDefault();
-        //     });
-
-        //     paramElement.get(0).addEventListener("drop", function(e : js.html.DragEvent) {
-        //         var toMoveIndex = Std.parseInt(e.dataTransfer.getData(parametersList.getDragKeyName()));
-        //         paramElement.toggleClass("hovertop", false);
-        //         paramElement.toggleClass("hoverbot", false);
-        //         if (paramIndex == null)
-        //             return;
-        //         var after = isAfter(e);
-        //         execMoveParameterTo(toMoveIndex, paramIndex, after);
-        //     });
-
-
-        //     var content = new Element("<content></content>").appendTo(paramElement);
-        //     var props = new Element("<ul>").appendTo(content);
-        //     if (previewAnimation != null) {
-        //         var slider = new Element('<li><dd>Preview</dd><input type="range" min="-1.0" max="1.0" step="0.01" value="${param.runtimeValue}"></input></li>').appendTo(props).find("input");
-        //         var range = new hide.comp.Range(null,slider);
-
-        //         range.setOnChangeUndo(undo, () -> param.runtimeValue, (v:Float) -> {
-        //             param.runtimeValue = v;
-        //             var runtimeParam = previewAnimation.parameterMap.get(param.name);
-        //             if (runtimeParam != null) {
-        //                 runtimeParam.runtimeValue = param.runtimeValue;
-        //             }
-        //         });
-
-        //         var def = new Element('<li><dd>Default</dd><input type="range" min="-1.0" max="1.0" step="0.01" value="${param.defaultValue}"></input></li>').appendTo(props).find("input");
-        //         var range = new hide.comp.Range(null,def);
-        //         range.setOnChangeUndo(undo, () -> param.defaultValue, (v:Float) -> param.defaultValue = v);
-        //     }
-
-        //     paramElement.find("header").get(0).addEventListener("contextmenu", function (e : js.html.MouseEvent) {
-        //         e.preventDefault();
-        //         hide.comp.ContextMenu.createFromEvent(e, [
-        //             {label: "Delete", click: () -> execRemoveParam(paramIndex)}
-        //         ]);
-        //     });
-
-        //     var menu = paramElement.find(".menu");
-        //     menu.on("click", (e) -> {
-        //         e.preventDefault();
-        //         hide.comp.ContextMenu.createDropdown(menu.get(0), [
-        //             {label: "Delete", click: () -> execRemoveParam(paramIndex)}
-        //         ]);
-        //     });
-        // }
-
         scenePreview.onObjectLoaded = () -> {
             setPreview(cast animGraph.nodes.find((f) -> Std.downcast(f, hrt.animgraph.nodes.Output) != null));
         }

hrt/prefab/Curve.hx

@@ -799,8 +799,7 @@ class Curve extends Prefab {
 			curveOrVal(x, defVal),
 			curveOrVal(y, defVal),
 			curveOrVal(z, defVal),
-			curveOrVal(w, 1.0)
-		);
+			curveOrVal(w, 1.0));
 	}
 
 	public static function getColorValue(curves: Array<Curve>) : Value {

hrt/shgraph/NodeGenContext.hx

@@ -128,7 +128,7 @@ class NodeGenContext {
 		var type = ShaderGraph.sgTypeToType(graphVar.type);
 		var variable = MapUtils.getOrPut(shaderVariables, id, {
 			var varId = hxsl.Ast.Tools.allocVarId();
-			var name = if (graphVar.isGlobal) ${graphVar.name} else '_local_${graphVar.name}_$varId';
+			var name = if (graphVar.isGlobal) graphVar.name else '_local_${graphVar.name}_$varId';
 			{variable: {id: varId, name: name, type: type, kind: Local}, isInit: false}
 		});
 		if (init != null && !variable.isInit) {