Outline highlighted connections

Author: williamchange

export_presets.cfg

@@ -1,5 +1,6 @@
 extends MMGraphNodeMinimal
 
+class_name MMGraphReroute
 
 const PREVIEW_SIZES : Array[int] = [ 0, 64, 128, 192]
 

material_maker/nodes/reroute/reroute.gd

@@ -40,6 +40,8 @@ var undoredo_move_node_selection_changed : bool = true
 enum ConnectionStyle {DIRECT, BEZIER, ROUNDED, MANHATTAN, DIAGONAL}
 var connection_line_style : int = ConnectionStyle.BEZIER
 
+var active_connections : Array[Dictionary]
+
 @onready var drag_cut_cursor = preload("res://material_maker/icons/knife.png")
 var connections_to_cut : Array[Dictionary]
 var drag_cut_line : PackedVector2Array
@@ -139,6 +141,7 @@ func _gui_input(event) -> void:
 		if connections_to_cut.size():
 			on_cut_connections(connections_to_cut)
 			connections_to_cut.clear()
+			update_active_connections()
 		Input.set_custom_mouse_cursor(null)
 		drag_cut_line.clear()
 		conns.clear()
@@ -272,6 +275,7 @@ func get_padded_node_rect(graph_node:GraphNode) -> Rect2:
 func _draw() -> void:
 	if drag_cut_line.size() > 1:
 		draw_polyline(drag_cut_line, get_theme_color("connection_knife", "GraphEdit"), 1.0)
+	$OutlineOverlay.queue_redraw()
 
 
 # Misc. useful functions
@@ -360,6 +364,7 @@ func on_cut_connections(connections_to_cut : Array):
 		var from_gen = get_node(str(c.from_node)).generator
 		var to_gen = get_node(str(c.to_node)).generator
 		if do_disconnect_node(c.from_node, c.from_port, c.to_node, c.to_port):
+			remove_active_connection(c.from_node, c.from_port, c.to_node, c.to_port)
 			conns.append({from=from_gen.name,from_port=c.from_port, to=to_gen.name, to_port=c.to_port})
 	var undo_actions = [
 		{ type="add_to_graph", parent=generator_hier_name, generators=[], connections=conns }
@@ -369,7 +374,6 @@ func on_cut_connections(connections_to_cut : Array):
 	]
 	undoredo.add("Cut node connections", undo_actions, redo_actions)
 
-
 func on_disconnect_node(from : String, from_slot : int, to : String, to_slot : int) -> void:
 	var from_gen = get_node(from).generator
 	var to_gen = get_node(to).generator
@@ -383,6 +387,7 @@ func on_disconnect_node(from : String, from_slot : int, to : String, to_slot : i
 			{ type="remove_connections", parent=generator_hier_name, connections=[connection] }
 		]
 		undoredo.add("Disconnect nodes", undo_actions, redo_actions)
+	remove_active_connection(from, from_slot, to, to_slot)
 
 func on_connections_changed(removed_connections : Array, added_connections : Array) -> void:
 	for c in removed_connections:
@@ -950,14 +955,20 @@ func _on_ButtonTransmitsSeed_toggled(button_pressed) -> void:
 
 var highlighting_connections : bool = false
 
+func update_active_connections() -> void:
+	active_connections.clear()
+	for c in get_connection_list():
+		if get_node(NodePath(c.from_node)).selected or get_node(NodePath(c.to_node)).selected:
+			active_connections.append(c)
+	$OutlineOverlay.queue_redraw()
+
 func highlight_connections() -> void:
 	if highlighting_connections:
 		return
 	highlighting_connections = true
 	while Input.is_mouse_button_pressed(MOUSE_BUTTON_LEFT):
 		await get_tree().process_frame
-	for c in get_connection_list():
-		set_connection_activity(c.from_node, c.from_port, c.to_node, c.to_port, 1.0 if get_node(NodePath(c.from_node)).selected or get_node(NodePath(c.to_node)).selected else 0.0)
+	update_active_connections()
 	highlighting_connections = false
 
 func _on_GraphEdit_node_selected(node : GraphElement) -> void:
@@ -1197,6 +1208,8 @@ func undoredo_command(command : Dictionary) -> void:
 		_:
 			print("Unknown undo/redo command:")
 			print(command)
+	await get_tree().process_frame
+	update_active_connections()
 
 func undoredo_move_node(node_name : String, old_pos : Vector2, new_pos : Vector2):
 	if old_pos == new_pos:
@@ -1673,3 +1686,12 @@ func color_comment_nodes() -> void:
 		picker.popup_hide.connect(picker.queue_free)
 		picker.popup_hide.connect(undoredo.end_group)
 		picker.popup()
+
+func remove_active_connection(from : String, from_slot : int, to : String, to_slot : int) -> void:
+	for c in len(active_connections):
+		var connection : Dictionary = active_connections[c]
+		if (connection.from_node == from and connection.from_port == from_slot
+				and connection.to_node == to and connection.to_port == to_slot):
+			active_connections.remove_at(c)
+			break
+	$OutlineOverlay.queue_redraw()

material_maker/panels/graph_edit/graph_edit.gd

@@ -1,9 +1,10 @@
-[gd_scene load_steps=8 format=3 uid="uid://dy1u50we7gtru"]
+[gd_scene load_steps=9 format=3 uid="uid://dy1u50we7gtru"]
 
 [ext_resource type="Script" uid="uid://dkp4w3at1o6cm" path="res://material_maker/panels/graph_edit/graph_edit.gd" id="1"]
 [ext_resource type="Texture2D" uid="uid://c0j4px4n72di5" path="res://material_maker/icons/icons.tres" id="2"]
 [ext_resource type="Script" uid="uid://bne3k0g56crmy" path="res://material_maker/tools/undo_redo/undo_redo.gd" id="3"]
 [ext_resource type="PackedScene" uid="uid://buj231c2gxm4o" path="res://material_maker/widgets/desc_button/desc_button.tscn" id="4"]
+[ext_resource type="Script" uid="uid://cbvroxeqaaopc" path="res://material_maker/panels/graph_edit/outline_overlay.gd" id="5_u5byk"]
 
 [sub_resource type="AtlasTexture" id="3"]
 atlas = ExtResource("2")
@@ -26,7 +27,6 @@ offset_right = 1529.0
 offset_bottom = 1193.0
 grow_horizontal = 2
 grow_vertical = 2
-scroll_offset = Vector2(-1, -1)
 grid_pattern = 1
 right_disconnects = true
 zoom_min = 0.25
@@ -86,6 +86,12 @@ layout_mode = 2
 [node name="UndoRedo" type="Node" parent="."]
 script = ExtResource("3")
 
+[node name="OutlineOverlay" type="Control" parent="."]
+anchors_preset = 0
+offset_right = 40.0
+offset_bottom = 40.0
+script = ExtResource("5_u5byk")
+
 [connection signal="connection_from_empty" from="." to="." method="request_popup" binds= [true]]
 [connection signal="connection_request" from="." to="." method="on_connect_node"]
 [connection signal="connection_to_empty" from="." to="." method="request_popup" binds= [false]]
@@ -95,7 +101,6 @@ script = ExtResource("3")
 [connection signal="node_deselected" from="." to="." method="_on_GraphEdit_node_unselected"]
 [connection signal="node_selected" from="." to="." method="_on_GraphEdit_node_selected"]
 [connection signal="paste_nodes_request" from="." to="." method="paste"]
-[connection signal="visibility_changed" from="." to="." method="_on_GraphEdit_visibility_changed"]
 [connection signal="timeout" from="Timer" to="." method="do_send_changed_signal"]
 [connection signal="text_changed" from="GraphUI/SubGraphUI/Label" to="." method="_on_Label_text_changed"]
 [connection signal="descriptions_changed" from="GraphUI/SubGraphUI/Description" to="." method="_on_Description_descriptions_changed"]

material_maker/panels/graph_edit/graph_edit.tscn

@@ -0,0 +1,47 @@
+extends Control
+
+# Connection lines outline overlay
+
+var should_draw_overlay : bool = true
+
+func _ready() -> void:
+	# Move behind connection lines to make outlines visible in comment nodes
+	owner.child_order_changed.connect(_move_behind_connections.call_deferred)
+
+	owner.begin_node_move.connect(func(): should_draw_overlay = false)
+	owner.end_node_move.connect(func():should_draw_overlay = true)
+
+
+func _move_behind_connections() -> void:
+	owner.move_child(self,owner.get_node("_connection_layer").get_index()-1)
+
+
+func _draw() -> void:
+	var reroute_positions : PackedVector2Array
+	var reroute_colors : PackedColorArray
+	
+	var zoom : float = owner.zoom
+	if owner.active_connections.size() and should_draw_overlay:
+		for line in owner.active_connections:
+			if owner.has_node(NodePath(line.to_node)) and owner.has_node(NodePath(line.to_node)):
+				var from_node : GraphNode = owner.get_node(NodePath(line.from_node))
+				var to_node : GraphNode = owner.get_node(NodePath(line.to_node))
+				if from_node and to_node:
+					var from_pos : Vector2 = from_node.get_output_port_position(line.from_port)*zoom + from_node.position
+					var to_pos : Vector2 = to_node.get_input_port_position(line.to_port)*zoom + to_node.position
+
+					var line_width : float = max(10, owner.connection_lines_thickness)
+					line_width = line_width + zoom if zoom < 0.9 else line_width * zoom
+					draw_polyline(owner._get_connection_line(from_pos, to_pos),
+							to_node.get_input_port_color(line.to_port).darkened(0.35),
+							 line_width, true)
+
+					var reroute_highlight_color : Color = from_node.get_output_port_color(line.from_port).darkened(0.35)
+					for node in [from_node, to_node]:
+						if node is MMGraphReroute and node.get_rect().get_center() not in reroute_positions:
+							reroute_positions.append(node.get_rect().get_center())
+							reroute_colors.append(reroute_highlight_color)
+
+		# draw reroute highlights
+		for p in len(reroute_positions):
+			draw_circle(reroute_positions[p], 20.0*zoom, reroute_colors[p], true, -1, true)

material_maker/panels/graph_edit/outline_overlay.gd

@@ -0,0 +1 @@
+uid://cbvroxeqaaopc