Update from upstream

Author: syntaxerror247

.gitignore

@@ -1,7 +1,7 @@
-# Godot 4+ specific ignores
+# Godot-specific ignores
 .godot/
-.DS_Store
+.nomedia
 
-# Imported translations (automatically generated from CSV files)
-*.translation
+# Others
 .~lock.*
+.DS_Store

CONTRIBUTING.md

@@ -8,7 +8,7 @@ Contributions don't need to be perfect, but they must move GodSVG in the right d
 
 GodSVG is made in Godot using its GDScript language. Refer to the [README](https://github.com/MewPurPur/GodSVG?tab=readme-ov-file#how-to-get-it) on how to get GodSVG running.
 
-Git must be configured, then you can clone the repository to your local machine: `git clone git@github.com:MewPurPur/GodSVG.git`
+Git must be configured, then you can clone the repository to your local machine: `git clone https://github.com/MewPurPur/GodSVG.git`
 
 The documentation won't go into detail about how to use Git. Refer to outside resources if you are unfamiliar with it.
 

assets/icons/Expand.svg

@@ -0,0 +1,37 @@
+[remap]
+
+importer="texture"
+type="CompressedTexture2D"
+uid="uid://0xptbhyrvsk1"
+path="res://.godot/imported/Expand.svg-54a09277a628926ed67d8eb42ef8926e.ctex"
+metadata={
+"vram_texture": false
+}
+
+[deps]
+
+source_file="res://assets/icons/Expand.svg"
+dest_files=["res://.godot/imported/Expand.svg-54a09277a628926ed67d8eb42ef8926e.ctex"]
+
+[params]
+
+compress/mode=0
+compress/high_quality=false
+compress/lossy_quality=0.7
+compress/hdr_compression=1
+compress/normal_map=0
+compress/channel_pack=0
+mipmaps/generate=false
+mipmaps/limit=-1
+roughness/mode=0
+roughness/src_normal=""
+process/fix_alpha_border=true
+process/premult_alpha=false
+process/normal_map_invert_y=false
+process/hdr_as_srgb=false
+process/hdr_clamp_exposure=false
+process/size_limit=0
+detect_3d/compress_to=1
+svg/scale=1.0
+editor/scale_with_editor_scale=false
+editor/convert_colors_with_editor_theme=false

assets/icons/Expand.svg.import

@@ -27,7 +27,8 @@ func pathdata_tests(print_success := false) -> bool:
 	"M 0 0 z 2 3": [["M", 0.0, 0.0], ["z"]],
 	"M3e1 4e-2": [["M", 3e1, 4e-2]],
 	"M5,1 A4,4,0,1,1,5,9": [["M", 5.0, 1.0], ["A", 4.0, 4.0, 0.0, 1, 1, 5.0, 9.0]],
-	"M4 1 2 - 4 4z": [["M", 4.0, 1.0]]
+	"M4 1 2 - 4 4z": [["M", 4.0, 1.0]],
+	"M1 6.9e-1": [["M", 1.0, 0.69]],
 	}
 
 	var tests_passed := true

assets/icons/PresetGrayscale.svg

@@ -2,6 +2,17 @@
 @tool
 extends EditorScript
 
+const COMMENTS_DICT = {
+	"Viewport": "The viewport is the area where the graphic is displayed. In similar applications, it's often called the canvas.",
+	"CDATA color": "CDATA shouldn't be translated. It's a type of XML section.",
+	"Editor formatter": "Refers to the formatter used for GodSVG's code editor.",
+	"Export formatter": "Refers to the formatter used when exporting.",
+	"Handle colors": "Refers to the colors of the draggable handles.",
+	"Handle size": "Refers to the size of the draggable handles.",
+	"Excluded": "Refers to the zero, one, or multiple UI parts to not be shown in the final layout. It's of plural cardinality.",
+	"Update check failed": "When checking for updates.",
+}
+
 const TRANSLATIONS_DIR = "translations"
 
 const HEADER = """#, fuzzy
@@ -80,11 +91,15 @@ func search_directory(dir: String) -> void:
 
 
 func update_translations() -> void:
+	var used_comments := PackedStringArray()
 	var location := ProjectSettings.globalize_path(TRANSLATIONS_DIR + "/GodSVG.pot")
 	var fa := FileAccess.open(location, FileAccess.WRITE)
 	fa.store_string(HEADER)
 
 	for msg in messages:
+		if COMMENTS_DICT.has(msg.msgid):
+			fa.store_string("#. %s\n" % COMMENTS_DICT[msg.msgid])
+			used_comments.append(msg.msgid)
 		fa.store_string(msg.to_string())
 	fa = null
 	print("Created " + TRANSLATIONS_DIR + "/GodSVG.pot with %d strings" % (messages.size() + 1))
@@ -105,3 +120,7 @@ func update_translations() -> void:
 				print("Updated " + TRANSLATIONS_DIR + "/%s: %s" % [file, output[0].rstrip("\n")])
 			else:
 				print("Updated " + TRANSLATIONS_DIR + "%s" % file)
+	
+	for id in COMMENTS_DICT:
+		if not used_comments.has(id):
+			print_rich("[color=#f66]The \"%s\" string, which has a comment defined for it, wasn't encountered." % id)

godot_only/scripts/tests.gd

@@ -336,11 +336,19 @@ toggle_snap={
 "events": [Object(InputEventKey,"resource_local_to_scene":false,"resource_name":"","device":-1,"window_id":0,"alt_pressed":true,"shift_pressed":false,"ctrl_pressed":false,"meta_pressed":false,"pressed":false,"keycode":83,"physical_keycode":0,"key_label":0,"unicode":115,"location":0,"echo":false,"script":null)
 ]
 }
+toggle_fullscreen={
+"deadzone": 0.2,
+"events": [Object(InputEventKey,"resource_local_to_scene":false,"resource_name":"","device":-1,"window_id":0,"alt_pressed":false,"shift_pressed":false,"ctrl_pressed":false,"meta_pressed":false,"pressed":false,"keycode":4194342,"physical_keycode":0,"key_label":0,"unicode":0,"location":0,"echo":false,"script":null)
+, Object(InputEventKey,"resource_local_to_scene":false,"resource_name":"","device":-1,"window_id":0,"alt_pressed":true,"shift_pressed":false,"ctrl_pressed":false,"meta_pressed":false,"pressed":false,"keycode":4194309,"physical_keycode":0,"key_label":0,"unicode":0,"location":0,"echo":false,"script":null)
+]
+}
 
 [input_devices]
 
 pointing/android/enable_long_press_as_right_click=true
 pointing/android/enable_pan_and_scale_gestures=true
+pen_tablet/driver="dummy"
+pen_tablet/driver.windows="dummy"
 
 [internationalization]
 

godot_only/scripts/update_translations.gd

@@ -223,6 +223,7 @@ func _unhandled_input(event: InputEvent) -> void:
 		if ShortcutUtils.is_action_pressed(event, action):
 			match action:
 				"quit": prompt_quit()
+				"toggle_fullscreen": toggle_fullscreen()
 				"about_info": open_about()
 				"about_donate": open_donate()
 				"check_updates": open_update_checker()
@@ -401,6 +402,31 @@ func prompt_quit() -> void:
 			Translator.translate("Do you want to quit GodSVG?"),
 			Translator.translate("Quit"), get_tree().quit)
 
+
+var was_window_maximized: bool
+var window_old_rect: Rect2
+
+func toggle_fullscreen() -> void:
+	if DisplayServer.window_get_mode() != DisplayServer.WindowMode.WINDOW_MODE_FULLSCREEN:
+		if DisplayServer.window_get_mode() == DisplayServer.WindowMode.WINDOW_MODE_MAXIMIZED:
+			was_window_maximized = true
+		else:
+			was_window_maximized = false
+			window_old_rect = Rect2(DisplayServer.window_get_position(),
+					DisplayServer.window_get_size())
+		DisplayServer.window_set_mode(DisplayServer.WindowMode.WINDOW_MODE_FULLSCREEN)
+	else:
+		if was_window_maximized:
+			DisplayServer.window_set_mode(DisplayServer.WindowMode.WINDOW_MODE_MAXIMIZED)
+		else:
+			DisplayServer.window_set_mode(DisplayServer.WindowMode.WINDOW_MODE_WINDOWED)
+			DisplayServer.window_set_size(window_old_rect.size)
+			# TODO Without at least 3 frames of wait, on my laptop the window would
+			# sometimes go a little higher than before after setting its position.
+			for i in 3:
+				await get_tree().process_frame
+			DisplayServer.window_set_position(window_old_rect.position)
+
 func open_update_checker() -> void:
 	remove_all_menus()
 	var confirmation_dialog := ConfirmDialogScene.instantiate()

project.godot

@@ -196,7 +196,7 @@ func get_export_text() -> String:
 signal hover_changed
 signal selection_changed
 
-signal requested_scroll_to_element_editor(xid: PackedInt32Array, inner_idx: int)
+signal requested_scroll_to_selection(xid: PackedInt32Array, inner_idx: int)
 
 # The viewport listens for this signal to put you in handle-placing mode.
 signal handle_added
@@ -705,10 +705,10 @@ func _move_selected(down: bool) -> void:
 		#xnode.get_attribute("d").move_subpath(inner_selections[0], down)
 	queue_svg_save()
 
-func view_in_list(xid: PackedInt32Array, inner_index := -1) -> void:
+func view_in_inspector(xid: PackedInt32Array, inner_index := -1) -> void:
 	if xid.is_empty():
 		return
-	requested_scroll_to_element_editor.emit(xid, inner_index)
+	requested_scroll_to_selection.emit(xid, inner_index)
 
 func duplicate_selected() -> void:
 	root_element.duplicate_xnodes(selected_xids)
@@ -761,7 +761,7 @@ func get_selection_context(popup_method: Callable, context: Utils.LayoutPart) ->
 					can_move_down = true
 		if context == Utils.LayoutPart.VIEWPORT:
 			btn_arr.append(ContextPopup.create_button(Translator.translate("View in Inspector"),
-					view_in_list.bind(selected_xids[0]), false,
+					view_in_inspector.bind(selected_xids[0]), false,
 					load("res://assets/icons/Inspector.svg")))
 
 		btn_arr.append(ContextPopup.create_shortcut_button("duplicate"))
@@ -791,7 +791,7 @@ func get_selection_context(popup_method: Callable, context: Utils.LayoutPart) ->
 				if idx < inner_idx:
 					inner_idx = idx
 			btn_arr.append(ContextPopup.create_button(Translator.translate("View in Inspector"),
-					view_in_list.bind(semi_selected_xid, inner_idx), false,
+					view_in_inspector.bind(semi_selected_xid, inner_idx), false,
 					load("res://assets/icons/Inspector.svg")))
 		match element_ref.name:
 			"path":
@@ -886,8 +886,12 @@ func popup_insert_command_after_context(popup_method: Callable) -> void:
 	# Disable invalid commands. Z is syntactically invalid, so disallow it even harder.
 	var warned_commands: PackedStringArray
 	var disabled_commands: PackedStringArray
+	# S commands are deliberately warned against in most cases, even though
+	# there is some sense in using them without a C or S command before them.
+	# Same for T commands in most cases, even though
+	# there is a notion of letting them determine the next shorthand quadratic curve.
 	match cmd_char.to_upper():
-		"M": warned_commands = PackedStringArray(["M", "Z", "T"])
+		"M": warned_commands = PackedStringArray(["M", "Z", "S", "T"])
 		"L", "H", "V", "A": warned_commands = PackedStringArray(["S", "T"])
 		"C", "S": warned_commands = PackedStringArray(["T"])
 		"Q", "T": warned_commands = PackedStringArray(["S"])