-
-
Notifications
You must be signed in to change notification settings - Fork 20.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[EditorInspector] Fix ImportDock and ProjectExport capitalize #21234
[EditorInspector] Fix ImportDock and ProjectExport capitalize #21234
Conversation
editor/import_dock.cpp
Outdated
@@ -423,6 +423,7 @@ ImportDock::ImportDock() { | |||
import_opts = memnew(EditorInspector); | |||
add_child(import_opts); | |||
import_opts->set_v_size_flags(SIZE_EXPAND_FILL); | |||
import_opts->set_enable_capitalize_paths(bool(EDITOR_GET("interface/inspector/capitalize_properties"))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CC @akien-mga Maybe we should rename this path interface/inspector/capitalize_properties
to a more generic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or should I use a different names for this? or maybe this can be the default to true instead of use EDITOR_GET
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should default to true in EditorInspector.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed.
791a839
to
da40211
Compare
da40211
to
50326a0
Compare
@@ -1536,7 +1536,7 @@ void EditorInspector::update_tree() { | |||
|
|||
Color c = sscolor; | |||
c.a /= level; | |||
section->setup(path_name, path_name, object, c, use_folding); | |||
section->setup(acc_path, path_name, object, c, use_folding); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix #21216 (comment)