aboutsummaryrefslogtreecommitdiff
path: root/game/src/OptionMenu
diff options
context:
space:
mode:
Diffstat (limited to 'game/src/OptionMenu')
-rw-r--r--game/src/OptionMenu/MonitorDisplaySelector.gd18
-rw-r--r--game/src/OptionMenu/OptionsMenu.gd16
-rw-r--r--game/src/OptionMenu/ResolutionSelector.gd69
-rw-r--r--game/src/OptionMenu/ScreenModeSelector.gd12
-rw-r--r--game/src/OptionMenu/SettingNodes/SettingHSlider.gd13
-rw-r--r--game/src/OptionMenu/SettingNodes/SettingOptionButton.gd40
-rw-r--r--game/src/OptionMenu/VideoTab.tscn1
-rw-r--r--game/src/OptionMenu/VolumeGrid.gd20
8 files changed, 114 insertions, 75 deletions
diff --git a/game/src/OptionMenu/MonitorDisplaySelector.gd b/game/src/OptionMenu/MonitorDisplaySelector.gd
index f2f0dc8..b665658 100644
--- a/game/src/OptionMenu/MonitorDisplaySelector.gd
+++ b/game/src/OptionMenu/MonitorDisplaySelector.gd
@@ -2,13 +2,17 @@ extends SettingOptionButton
func _setup_button():
clear()
- for screen_index in range(DisplayServer.get_screen_count()):
+ for screen_index in DisplayServer.get_screen_count():
add_item("Monitor %d" % (screen_index + 1))
default_selected = get_viewport().get_window().current_screen
-func _on_item_selected(index):
- var window := get_viewport().get_window()
- var mode := window.mode
- window.mode = Window.MODE_WINDOWED
- get_viewport().get_window().set_current_screen(index)
- window.mode = mode
+func _on_item_selected(index : int):
+ if _valid_index(index):
+ var window := get_viewport().get_window()
+ var mode := window.mode
+ window.mode = Window.MODE_WINDOWED
+ get_viewport().get_window().set_current_screen(index)
+ window.mode = mode
+ else:
+ push_error("Invalid MonitorDisplaySelector index: %d" % index)
+ reset_setting()
diff --git a/game/src/OptionMenu/OptionsMenu.gd b/game/src/OptionMenu/OptionsMenu.gd
index 5aba7f2..c25c3b8 100644
--- a/game/src/OptionMenu/OptionsMenu.gd
+++ b/game/src/OptionMenu/OptionsMenu.gd
@@ -38,21 +38,23 @@ func toggle_locale_button_visibility(locale_visible : bool):
$LocaleVBox/LocaleHBox/LocaleButton.visible = locale_visible
func _on_back_button_pressed():
- Events.Options.save_settings_from_file()
+ Events.Options.save_settings_to_file()
back_button_pressed.emit()
func _on_window_close_requested() -> void:
if visible:
- Events.Options.save_settings_from_file()
+ Events.Options.save_settings_to_file()
func _save_overrides() -> void:
- var override_path := ProjectSettings.get_setting("application/config/project_settings_override") as String
- if override_path == null or override_path.is_empty():
- override_path = ProjectSettings.get_setting("openvic2/settings/settings_file_path") as String
+ var override_path : String = ProjectSettings.get_setting("application/config/project_settings_override", "")
+ if override_path.is_empty():
+ override_path = ProjectSettings.get_setting(Events.Options.settings_file_path_setting, Events.Options.settings_file_path_default)
var file := ConfigFile.new()
- file.load(override_path)
+ var err_ret := file.load(override_path)
+ if err_ret != OK: push_error("Failed to load overrides from %s" % override_path)
file.set_value("display", "window/size/mode", get_viewport().get_window().mode)
var resolution : Vector2i = Resolution.get_current_resolution()
file.set_value("display", "window/size/viewport_width", resolution.x)
file.set_value("display", "window/size/viewport_height", resolution.y)
- file.save(override_path)
+ err_ret = file.save(override_path)
+ if err_ret != OK: push_error("Failed to save overrides to %s" % override_path)
diff --git a/game/src/OptionMenu/ResolutionSelector.gd b/game/src/OptionMenu/ResolutionSelector.gd
index e602bab..6cdaca8 100644
--- a/game/src/OptionMenu/ResolutionSelector.gd
+++ b/game/src/OptionMenu/ResolutionSelector.gd
@@ -1,58 +1,65 @@
extends SettingOptionButton
@export
-var default_value : Vector2i = Vector2i(-1, -1)
+var default_value : Vector2i = Resolution.error_resolution
-func add_resolution(value : Vector2i, selection_name : String = "") -> void:
- if selection_name.is_empty():
- selection_name = "%sx%s" % [value.x, value.y]
- add_item(selection_name)
- set_item_metadata(item_count - 1, value)
-
-func find_resolution_value(value : Vector2i) -> int:
- for item_index in range(item_count):
+func _find_resolution_index_by_value(value : Vector2i) -> int:
+ for item_index in item_count:
if get_item_metadata(item_index) == value:
return item_index
return -1
-func _setup_button():
- if default_value.x < 0:
- default_value.x = ProjectSettings.get_setting("display/window/size/viewport_width")
-
- if default_value.y < 0:
- default_value.y = ProjectSettings.get_setting("display/window/size/viewport_height")
-
+func _sync_resolutions(to_select : Vector2i = Resolution.get_current_resolution()) -> void:
clear()
default_selected = -1
selected = -1
- for resolution in Resolution.get_resolution_name_list():
- var resolution_value := Resolution.get_resolution(resolution)
- add_resolution(resolution_value, resolution)
+ for resolution_value in Resolution.get_resolution_value_list():
+ add_item(Resolution.get_resolution_display_name(resolution_value))
+ set_item_metadata(item_count - 1, resolution_value)
if resolution_value == default_value:
default_selected = item_count - 1
- if resolution_value == Resolution.get_current_resolution():
+ if resolution_value == to_select:
selected = item_count - 1
if default_selected == -1:
- add_resolution(default_value)
default_selected = item_count - 1
if selected == -1:
selected = default_selected
+func _setup_button():
+ if default_value.x <= 0:
+ default_value.x = ProjectSettings.get_setting("display/window/size/viewport_width")
+ if default_value.y <= 0:
+ default_value.y = ProjectSettings.get_setting("display/window/size/viewport_height")
+ Resolution.add_resolution(default_value, &"default")
+ _sync_resolutions()
+
func _get_value_for_file(select_value : int):
- return get_item_metadata(select_value)
+ if _valid_index(select_value):
+ return get_item_metadata(select_value)
+ else:
+ return null
func _set_value_from_file(load_value):
- var resolution_value := load_value as Vector2i
- selected = find_resolution_value(resolution_value)
- if selected == -1:
- if add_nonstandard_value:
- add_resolution(resolution_value)
- selected = item_count - 1
- else: push_error("Setting value '%s' invalid for setting [%s] %s" % [load_value, section_name, setting_name])
+ var target_resolution := Resolution.error_resolution
+ match typeof(load_value):
+ TYPE_VECTOR2I: target_resolution = load_value
+ TYPE_STRING, TYPE_STRING_NAME: target_resolution = Resolution.get_resolution_value_from_string(load_value)
+ if target_resolution != Resolution.error_resolution:
+ selected = _find_resolution_index_by_value(target_resolution)
+ if selected != -1: return
+ if Resolution.add_resolution(target_resolution):
+ _sync_resolutions(target_resolution)
+ return
+ push_error("Setting value '%s' invalid for setting [%s] %s" % [load_value, section_name, setting_name])
+ selected = default_selected
-func _on_item_selected(index):
- Resolution.set_resolution(get_item_metadata(index))
+func _on_item_selected(index : int):
+ if _valid_index(index):
+ Resolution.set_resolution(get_item_metadata(index))
+ else:
+ push_error("Invalid ResolutionSelector index: %d" % index)
+ reset_setting()
diff --git a/game/src/OptionMenu/ScreenModeSelector.gd b/game/src/OptionMenu/ScreenModeSelector.gd
index 92c5d60..b4fc5ab 100644
--- a/game/src/OptionMenu/ScreenModeSelector.gd
+++ b/game/src/OptionMenu/ScreenModeSelector.gd
@@ -29,7 +29,11 @@ func _setup_button():
selected = default_selected
func _on_item_selected(index : int):
- var window := get_viewport().get_window()
- var current_resolution := Resolution.get_current_resolution()
- window.mode = get_window_mode_from_screen_mode(index)
- Resolution.set_resolution(current_resolution)
+ if _valid_index(index):
+ var window := get_viewport().get_window()
+ var current_resolution := Resolution.get_current_resolution()
+ window.mode = get_window_mode_from_screen_mode(index)
+ Resolution.set_resolution(current_resolution)
+ else:
+ push_error("Invalid ScreenModeSelector index: %d" % index)
+ reset_setting()
diff --git a/game/src/OptionMenu/SettingNodes/SettingHSlider.gd b/game/src/OptionMenu/SettingNodes/SettingHSlider.gd
index cf2adf4..6a0e5ed 100644
--- a/game/src/OptionMenu/SettingNodes/SettingHSlider.gd
+++ b/game/src/OptionMenu/SettingNodes/SettingHSlider.gd
@@ -17,7 +17,18 @@ func _ready():
func load_setting(file : ConfigFile):
if file == null: return
- value = file.get_value(section_name, setting_name, default_value)
+ var load_value = file.get_value(section_name, setting_name, default_value)
+ match typeof(load_value):
+ TYPE_FLOAT, TYPE_INT:
+ value = load_value
+ return
+ TYPE_STRING, TYPE_STRING_NAME:
+ var load_string := load_value as String
+ if load_string.is_valid_float():
+ value = load_string.to_float()
+ return
+ push_error("Setting value '%s' invalid for setting [%s] \"%s\"" % [load_value, section_name, setting_name])
+ value = default_value
func save_setting(file : ConfigFile):
if file == null: return
diff --git a/game/src/OptionMenu/SettingNodes/SettingOptionButton.gd b/game/src/OptionMenu/SettingNodes/SettingOptionButton.gd
index 3a5c979..c7b760d 100644
--- a/game/src/OptionMenu/SettingNodes/SettingOptionButton.gd
+++ b/game/src/OptionMenu/SettingNodes/SettingOptionButton.gd
@@ -11,30 +11,39 @@ var setting_name : String = "SettingOptionMenu"
var default_selected : int = -1:
get: return default_selected
set(v):
- if v == -1:
+ if v < 0 or item_count == 0:
default_selected = -1
return
default_selected = v % item_count
-@export
-var add_nonstandard_value := false
+func _valid_index(index : int) -> bool:
+ return 0 <= index and index < item_count
func _get_value_for_file(select_value : int):
- if select_value > -1:
- return get_item_text(select_value)
+ if _valid_index(select_value):
+ return select_value
else:
return null
func _set_value_from_file(load_value) -> void:
- selected = -1
- for item_index in range(item_count):
- if load_value == get_item_text(item_index):
- selected = item_index
- if selected == -1:
- if add_nonstandard_value:
- add_item(load_value)
- selected = item_count - 1
- else: push_error("Setting value '%s' invalid for setting [%s] \"%s\"" % [load_value, section_name, setting_name])
+ match typeof(load_value):
+ TYPE_INT:
+ if _valid_index(load_value):
+ selected = load_value
+ return
+ TYPE_STRING, TYPE_STRING_NAME:
+ var load_string := load_value as String
+ if load_string.is_valid_int():
+ var load_int := load_string.to_int()
+ if _valid_index(load_int):
+ selected = load_int
+ return
+ for item_index in item_count:
+ if load_string == get_item_text(item_index):
+ selected = item_index
+ return
+ push_error("Setting value '%s' invalid for setting [%s] \"%s\"" % [load_value, section_name, setting_name])
+ selected = default_selected
func _setup_button() -> void:
pass
@@ -44,6 +53,9 @@ func _ready():
Events.Options.save_settings.connect(save_setting)
Events.Options.reset_settings.connect(reset_setting)
_setup_button()
+ if not _valid_index(default_selected) or selected == -1:
+ OS.alert("Failed to generate %s %s options." % [setting_name, section_name], "%s Options Error" % section_name)
+ get_tree().quit()
func load_setting(file : ConfigFile) -> void:
if file == null: return
diff --git a/game/src/OptionMenu/VideoTab.tscn b/game/src/OptionMenu/VideoTab.tscn
index d46f056..f14f757 100644
--- a/game/src/OptionMenu/VideoTab.tscn
+++ b/game/src/OptionMenu/VideoTab.tscn
@@ -34,7 +34,6 @@ popup/item_0/id = 0
script = ExtResource("1_i8nro")
section_name = "Video"
setting_name = "Resolution"
-add_nonstandard_value = true
[node name="ScreenModeLabel" type="Label" parent="VBoxContainer/GridContainer"]
layout_mode = 2
diff --git a/game/src/OptionMenu/VolumeGrid.gd b/game/src/OptionMenu/VolumeGrid.gd
index fae5ff6..8d5a016 100644
--- a/game/src/OptionMenu/VolumeGrid.gd
+++ b/game/src/OptionMenu/VolumeGrid.gd
@@ -1,8 +1,8 @@
extends GridContainer
-const RATIO_FOR_LINEAR = 100
+const RATIO_FOR_LINEAR : float = 100
-var _slider_dictionary := {}
+var _slider_dictionary : Dictionary
func get_db_as_volume_value(db : float) -> float:
# db_to_linear produces a float between 0 and 1 from a db value
@@ -12,7 +12,7 @@ func get_volume_value_as_db(value : float) -> float:
# linear_to_db consumes a float between 0 and 1 to produce the db value
return linear_to_db(value / RATIO_FOR_LINEAR)
-func add_volume_column(bus_name : StringName, bus_index : int) -> HSlider:
+func add_volume_row(bus_name : StringName, bus_index : int) -> HSlider:
var volume_label := Label.new()
volume_label.text = bus_name + " Volume"
add_child(volume_label)
@@ -33,22 +33,22 @@ func add_volume_column(bus_name : StringName, bus_index : int) -> HSlider:
func _ready():
for bus_index in AudioServer.bus_count:
- add_volume_column(AudioServer.get_bus_name(bus_index), bus_index)
+ add_volume_row(AudioServer.get_bus_name(bus_index), bus_index)
func _on_slider_value_changed(value : float, bus_index : int) -> void:
AudioServer.set_bus_volume_db(bus_index, get_volume_value_as_db(value))
func _on_options_menu_load_settings(load_file : ConfigFile):
- for volume_label_text in _slider_dictionary:
- _slider_dictionary[volume_label_text].load_setting(load_file)
+ for volume_slider in _slider_dictionary.values():
+ volume_slider.load_setting(load_file)
func _on_options_menu_save_settings(save_file : ConfigFile):
- for volume_label_text in _slider_dictionary:
- _slider_dictionary[volume_label_text].save_setting(save_file)
+ for volume_slider in _slider_dictionary.values():
+ volume_slider.save_setting(save_file)
func _on_options_menu_reset_settings():
- for volume_label_text in _slider_dictionary:
- _slider_dictionary[volume_label_text].reset_setting()
+ for volume_slider in _slider_dictionary.values():
+ volume_slider.reset_setting()