Skip to content

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Jun 11, 2025

Same as #107156, but instead moves editor/naming settings.
I also made setting compatibility more explicit.

@KoBeWi KoBeWi added this to the 4.x milestone Jun 11, 2025
@KoBeWi KoBeWi requested review from a team as code owners June 11, 2025 13:39
@Mickeon
Copy link
Member

Mickeon commented Jun 11, 2025

The settings' original descriptions need to be moved to the editor settings, but I gather they would completely lose their descriptions in this PR?

@Mickeon Mickeon requested a review from AThousandShips June 11, 2025 14:19
@KoBeWi KoBeWi force-pushed the settings_of_project_editor branch from a9c2d4a to 1250ebb Compare June 11, 2025 14:57
@KoBeWi KoBeWi requested a review from a team as a code owner June 11, 2025 14:57
Copy link
Member

@Mickeon Mickeon left a 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.

@KoBeWi KoBeWi force-pushed the settings_of_project_editor branch from 1250ebb to ebce97e Compare June 11, 2025 15:17
Copy link
Member

@Mickeon Mickeon left a 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);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit weird.

Copy link
Member Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants