Fix various issues regarding the responsive UI

- Instead of `DisplayServer.window_set_min_size` the project now uses
  `get_window().min_size = ...` everywhere (as the docs recommend).
- Overlay components are now checked for two meta property `x_padding`
  and `y_padding` which can be used to increase the minimum size of the
  window in cases where the component is bigger than the previous min.
- Each component `item_rect_changed` signal is now connected to the new
  `_update_min_size` function when opened, which ensures that the window
  will always big enough to contain the entire component.
- A label in the settings panel uses auto-wrapping, which currently
  leads to issues when the label has no custom_minimum_size, see:
  godotengine/godot#83546
  I opted for an alternative fix (as proposed by nathanfranke in the
  issue linked above) which involves setting the anchors and offset of
  the label to the TOP_WIDE preset.

Author: njamster

entry_point/entry_point.gd

@@ -17,7 +17,7 @@ func _ready() -> void:
 				change_to_main_scene.call_deferred()
 				return
 
-	DisplayServer.window_set_min_size(size)
+	get_window().min_size = size
 
 	%Path.text = Settings.store_path
 	%BrowseFiles.grab_focus()

main_window/main_window.gd

@@ -14,10 +14,10 @@ func _ready() -> void:
 	if Engine.is_editor_hint():
 		return
 
-	DisplayServer.window_set_min_size(Vector2i(
+	get_window().min_size = Vector2i(
 		# FIXME: get rid of those hardcoded values
 		2 * 28 + 2 * 16 + 160, 2 * minimum_vertical_margin + 79 + 5 * 40
-	))
+	)
 	get_tree().get_root().size_changed.connect(_on_window_size_changed)
 
 

main_window/overlay/calendar_widget/calendar_widget.gd

@@ -18,7 +18,9 @@ func update_month() -> void:
 
 	# remove old children
 	for child in $VBox/GridContainer.get_children():
-		child.queue_free()
+		# This *must* be free, not queue_free! Otherwise, it will blow up the vertical size of the
+		# panel since it's not fast enough and old and new nodes briefly co-exist.
+		child.free()
 
 	# add new children
 	for day_name in Date._DAY_NAMES:
@@ -76,3 +78,8 @@ func _on_today_pressed() -> void:
 func _on_next_month_pressed() -> void:
 	anchor_date = anchor_date.add_months(1)
 	update_month()
+
+
+func _on_item_rect_changed() -> void:
+	set_meta("x_padding", 2 * abs(offset_right))
+	set_meta("y_padding", 18 + offset_top)

main_window/overlay/calendar_widget/calendar_widget.tscn

@@ -66,6 +66,7 @@ text = "Next Month"
 layout_mode = 2
 columns = 7
 
+[connection signal="item_rect_changed" from="." to="." method="_on_item_rect_changed"]
 [connection signal="visibility_changed" from="." to="." method="_on_visibility_changed"]
 [connection signal="pressed" from="VBox/HBox/PreviousMonth" to="." method="_on_previous_month_pressed"]
 [connection signal="pressed" from="VBox/HBox/Today" to="." method="_on_today_pressed"]

main_window/overlay/overlay.gd

@@ -10,26 +10,34 @@ func _ready() -> void:
 	EventBus.settings_button_pressed.connect(open_component.bind($SettingsPanel, true))
 
 
-func open_component(component : Node, dimmed_background : bool) -> void:
+func open_component(component : Control, dimmed_background : bool) -> void:
 	$Background.color.a = 0.7 if dimmed_background else 0.0
 
 	self.show()
 	component.show()
 
 	_previous_min_size = DisplayServer.window_get_min_size()
-	DisplayServer.window_set_min_size(Vector2i(
-		max(component.size.x, _previous_min_size.x),
-		max(component.size.y, _previous_min_size.y)
-	))
+
+	component.item_rect_changed.connect(_update_min_size.bind(component))
+	_update_min_size(component)
+
+
+func _update_min_size(component : Control) -> void:
+	get_window().min_size = Vector2i(
+		max(component.size.x + component.get_meta("x_padding", 0), _previous_min_size.x),
+		max(component.size.y + component.get_meta("y_padding", 0), _previous_min_size.y)
+	)
 
 
 func close_overlay() -> void:
 	self.hide()
-	$CalendarWidget.hide()
-	$SettingsPanel.hide()
+	for component in [$CalendarWidget, $SettingsPanel]:
+		component.hide()
+		if component.item_rect_changed.is_connected(_update_min_size):
+			component.item_rect_changed.disconnect(_update_min_size)
 
 	# FIXME: re-triggering the min_size calculation of the MainWindow would probably be cleaner
-	DisplayServer.window_set_min_size(_previous_min_size)
+	get_window().min_size = _previous_min_size
 	_previous_min_size = Vector2i.ZERO
 
 

main_window/overlay/overlay.tscn

@@ -20,5 +20,9 @@ visible = false
 
 [node name="SettingsPanel" parent="." instance=ExtResource("2_qeh0g")]
 visible = false
+offset_left = -306.5
+offset_top = -150.5
+offset_right = 306.5
+offset_bottom = 150.5
 
 [connection signal="gui_input" from="Background" to="." method="_on_dimmed_background_gui_input"]

main_window/overlay/settings_panel/settings_panel.gd

@@ -1,6 +1,13 @@
 extends PanelContainer
 
 
+func _enter_tree() -> void:
+	# See: https://github.com/godotengine/godot/issues/83546#issuecomment-1856927502
+	# Without this line (or a custom_minimum_size for the label), the autowrapping will blow up the
+	# vertical size of the panel. This can (as of now) only be done via code, not in the editor.
+	$VBox/VBox1/Explanation.set_anchors_and_offsets_preset(Control.PRESET_TOP_WIDE)
+
+
 func _on_visibility_changed() -> void:
 	if visible:
 		%StorePath/Path.text = Settings.store_path

main_window/overlay/settings_panel/settings_panel.tscn

@@ -15,6 +15,8 @@ offset_bottom = 20.0
 grow_horizontal = 2
 grow_vertical = 2
 script = ExtResource("1_yrmyc")
+metadata/x_padding = 64
+metadata/y_padding = 64
 
 [node name="VBox" type="VBoxContainer" parent="."]
 layout_mode = 2