-
-
Notifications
You must be signed in to change notification settings - Fork 22.8k
Move editor naming settings out of project settings #107405
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
base: master
Are you sure you want to change the base?
Conversation
The settings' original descriptions need to be moved to the editor settings, but I gather they would completely lose their descriptions in this PR? |
a9c2d4a
to
1250ebb
Compare
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 have personal gripes with these descriptions but that's not the point of this PR.
1250ebb
to
ebce97e
Compare
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 can't say whether the code is good for this, but the docs are fine.
Do note, for the uninformed, that the functionality can be restored per-project because of #69012 . The PR adds an override if necessary
@@ -2203,6 +2227,8 @@ void EditorSettings::_bind_methods() { | |||
ClassDB::bind_method(D_METHOD("get_changed_settings"), &EditorSettings::get_changed_settings); | |||
ClassDB::bind_method(D_METHOD("mark_setting_changed", "setting"), &EditorSettings::mark_setting_changed); | |||
|
|||
ClassDB::bind_method(D_METHOD("_define_from_project_setting"), &EditorSettings::define_from_project_setting); |
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.
A bit weird.
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.
It's needed to use call()
, to avoid including the class.
Same as #107156, but instead moves
editor/naming
settings.I also made setting compatibility more explicit.