Skip to content

Commit 13eee43

Browse files
committed
Merge pull request #105072 from lodetrick/action-map-editor-scale
Fix OpenXR Action Map GUI not scaling according to Editor Scale
2 parents 0fbb22c + 0ead587 commit 13eee43

File tree

6 files changed

+19
-10
lines changed

6 files changed

+19
-10
lines changed

modules/openxr/editor/openxr_action_editor.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
#include "openxr_action_editor.h"
3232

3333
#include "editor/editor_string_names.h"
34+
#include "editor/themes/editor_scale.h"
3435

3536
void OpenXRActionEditor::_bind_methods() {
3637
ClassDB::bind_method(D_METHOD("_do_set_name", "name"), &OpenXRActionEditor::_do_set_name);
@@ -134,14 +135,14 @@ OpenXRActionEditor::OpenXRActionEditor(Ref<OpenXRAction> p_action) {
134135
action_name = memnew(LineEdit);
135136
action_name->set_text(action->get_name());
136137
action_name->set_tooltip_text(TTR("Internal name of the action. Some XR runtimes don't allow spaces or special characters."));
137-
action_name->set_custom_minimum_size(Size2(150.0, 0.0));
138+
action_name->set_custom_minimum_size(Size2(150.0 * EDSCALE, 0.0));
138139
action_name->connect(SceneStringName(text_changed), callable_mp(this, &OpenXRActionEditor::_on_action_name_changed));
139140
add_child(action_name);
140141

141142
action_localized_name = memnew(LineEdit);
142143
action_localized_name->set_text(action->get_localized_name());
143144
action_localized_name->set_tooltip_text(TTR("Human-readable name of the action. This can be displayed to end users."));
144-
action_localized_name->set_custom_minimum_size(Size2(150.0, 0.0));
145+
action_localized_name->set_custom_minimum_size(Size2(150.0 * EDSCALE, 0.0));
145146
action_localized_name->set_h_size_flags(Control::SIZE_EXPAND_FILL);
146147
action_localized_name->connect(SceneStringName(text_changed), callable_mp(this, &OpenXRActionEditor::_on_action_localized_name_changed));
147148
add_child(action_localized_name);
@@ -154,7 +155,7 @@ OpenXRActionEditor::OpenXRActionEditor(Ref<OpenXRAction> p_action) {
154155
action_type_button->add_item("Pose", OpenXRAction::OPENXR_ACTION_POSE);
155156
action_type_button->add_item("Haptic", OpenXRAction::OPENXR_ACTION_HAPTIC);
156157
action_type_button->select(int(action->get_action_type()));
157-
action_type_button->set_custom_minimum_size(Size2(100.0, 0.0));
158+
action_type_button->set_custom_minimum_size(Size2(100.0 * EDSCALE, 0.0));
158159
action_type_button->connect(SceneStringName(item_selected), callable_mp(this, &OpenXRActionEditor::_on_item_selected));
159160
add_child(action_type_button);
160161

modules/openxr/editor/openxr_action_map_editor.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
#include "editor/editor_node.h"
3535
#include "editor/gui/editor_bottom_panel.h"
3636
#include "editor/gui/editor_file_dialog.h"
37+
#include "editor/themes/editor_scale.h"
3738

3839
HashMap<String, String> OpenXRActionMapEditor::interaction_profile_editors;
3940
HashMap<String, String> OpenXRActionMapEditor::binding_modifier_editors;
@@ -431,7 +432,7 @@ String OpenXRActionMapEditor::get_binding_modifier_editor_class(const String &p_
431432

432433
OpenXRActionMapEditor::OpenXRActionMapEditor() {
433434
undo_redo = EditorUndoRedoManager::get_singleton();
434-
set_custom_minimum_size(Size2(0.0, 300.0));
435+
set_custom_minimum_size(Size2(0.0, 300.0 * EDSCALE));
435436

436437
top_hb = memnew(HBoxContainer);
437438
add_child(top_hb);

modules/openxr/editor/openxr_action_set_editor.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
#include "openxr_action_set_editor.h"
3232

3333
#include "editor/editor_string_names.h"
34+
#include "editor/themes/editor_scale.h"
3435
#include "openxr_action_editor.h"
3536

3637
void OpenXRActionSetEditor::_bind_methods() {
@@ -244,22 +245,22 @@ OpenXRActionSetEditor::OpenXRActionSetEditor(Ref<OpenXRActionMap> p_action_map,
244245
action_set_name = memnew(LineEdit);
245246
action_set_name->set_text(action_set->get_name());
246247
action_set_name->set_tooltip_text(TTR("Internal name of the action. Some XR runtimes don't allow spaces or special characters."));
247-
action_set_name->set_custom_minimum_size(Size2(150.0, 0.0));
248+
action_set_name->set_custom_minimum_size(Size2(150.0 * EDSCALE, 0.0));
248249
action_set_name->connect(SceneStringName(text_changed), callable_mp(this, &OpenXRActionSetEditor::_on_action_set_name_changed));
249250
action_set_hb->add_child(action_set_name);
250251

251252
action_set_localized_name = memnew(LineEdit);
252253
action_set_localized_name->set_text(action_set->get_localized_name());
253254
action_set_localized_name->set_tooltip_text(TTR("Human-readable name of the action set. This can be displayed to end users."));
254-
action_set_localized_name->set_custom_minimum_size(Size2(150.0, 0.0));
255+
action_set_localized_name->set_custom_minimum_size(Size2(150.0 * EDSCALE, 0.0));
255256
action_set_localized_name->set_h_size_flags(Control::SIZE_EXPAND_FILL);
256257
action_set_localized_name->connect(SceneStringName(text_changed), callable_mp(this, &OpenXRActionSetEditor::_on_action_set_localized_name_changed));
257258
action_set_hb->add_child(action_set_localized_name);
258259

259260
action_set_priority = memnew(TextEdit);
260261
action_set_priority->set_text(itos(action_set->get_priority()));
261262
action_set_priority->set_tooltip_text(TTR("Priority of the action set. If multiple action sets bind to the same input, the action set with the highest priority will be updated."));
262-
action_set_priority->set_custom_minimum_size(Size2(50.0, 0.0));
263+
action_set_priority->set_custom_minimum_size(Size2(50.0 * EDSCALE, 0.0));
263264
action_set_priority->connect(SceneStringName(text_changed), callable_mp(this, &OpenXRActionSetEditor::_on_action_set_priority_changed));
264265
action_set_hb->add_child(action_set_priority);
265266

modules/openxr/editor/openxr_binding_modifiers_dialog.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@
3232
#include "../action_map/openxr_interaction_profile_metadata.h"
3333
#include "openxr_action_map_editor.h"
3434

35+
#include "editor/themes/editor_scale.h"
36+
3537
void OpenXRBindingModifiersDialog::_bind_methods() {
3638
ClassDB::bind_method(D_METHOD("_do_add_binding_modifier_editor", "binding_modifier_editor"), &OpenXRBindingModifiersDialog::_do_add_binding_modifier_editor);
3739
ClassDB::bind_method(D_METHOD("_do_remove_binding_modifier_editor", "binding_modifier_editor"), &OpenXRBindingModifiersDialog::_do_remove_binding_modifier_editor);
@@ -191,7 +193,7 @@ OpenXRBindingModifiersDialog::OpenXRBindingModifiersDialog() {
191193
set_transient(true);
192194

193195
binding_modifier_sc = memnew(ScrollContainer);
194-
binding_modifier_sc->set_custom_minimum_size(Size2(350.0, 0.0));
196+
binding_modifier_sc->set_custom_minimum_size(Size2(350.0 * EDSCALE, 0.0));
195197
binding_modifier_sc->set_h_size_flags(Control::SIZE_EXPAND_FILL);
196198
binding_modifier_sc->set_v_size_flags(Control::SIZE_EXPAND_FILL);
197199
binding_modifier_sc->set_horizontal_scroll_mode(ScrollContainer::SCROLL_MODE_DISABLED);

modules/openxr/editor/openxr_select_action_dialog.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,8 @@
3030

3131
#include "openxr_select_action_dialog.h"
3232

33+
#include "editor/themes/editor_scale.h"
34+
3335
void OpenXRSelectActionDialog::_bind_methods() {
3436
ADD_SIGNAL(MethodInfo("action_selected", PropertyInfo(Variant::STRING, "action")));
3537
}
@@ -124,7 +126,7 @@ OpenXRSelectActionDialog::OpenXRSelectActionDialog(Ref<OpenXRActionMap> p_action
124126
set_title(TTR("Select an action"));
125127

126128
scroll = memnew(ScrollContainer);
127-
scroll->set_custom_minimum_size(Size2(600.0, 400.0));
129+
scroll->set_custom_minimum_size(Size2(600.0 * EDSCALE, 400.0 * EDSCALE));
128130
add_child(scroll);
129131

130132
main_vb = memnew(VBoxContainer);

modules/openxr/editor/openxr_select_interaction_profile_dialog.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@
3333
#include "../action_map/openxr_interaction_profile_metadata.h"
3434
#include "../openxr_api.h"
3535

36+
#include "editor/themes/editor_scale.h"
37+
3638
void OpenXRSelectInteractionProfileDialog::_bind_methods() {
3739
ADD_SIGNAL(MethodInfo("interaction_profile_selected", PropertyInfo(Variant::STRING, "interaction_profile")));
3840
}
@@ -116,7 +118,7 @@ OpenXRSelectInteractionProfileDialog::OpenXRSelectInteractionProfileDialog() {
116118
set_title(TTR("Select an interaction profile"));
117119

118120
scroll = memnew(ScrollContainer);
119-
scroll->set_custom_minimum_size(Size2(600.0, 400.0));
121+
scroll->set_custom_minimum_size(Size2(600.0 * EDSCALE, 400.0 * EDSCALE));
120122
add_child(scroll);
121123

122124
main_vb = memnew(VBoxContainer);

0 commit comments

Comments
 (0)