Skip to content
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

Hide redundant categories from project settings #61450

Merged
merged 1 commit into from
May 31, 2022

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented May 27, 2022

Salvage of #39670 with some extras. Aside from Autoloads, it hides Editor Plugins and Shader Globals:
image
(they have their own tabs too)

Fixes #39639
Fixes #24453

@KoBeWi KoBeWi added this to the 4.0 milestone May 27, 2022
@akien-mga akien-mga merged commit c1460b9 into godotengine:master May 31, 2022
@akien-mga
Copy link
Member

Thanks!

@akien-mga akien-mga added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label May 31, 2022
@KoBeWi KoBeWi deleted the autohide branch May 31, 2022 10:30
@akien-mga
Copy link
Member

Cherry-picked for 3.5.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label May 31, 2022
@KoBeWi
Copy link
Member Author

KoBeWi commented Jun 6, 2022

This could've been done better actually. Inspector will automatically hide properties that don't have PROPERTY_USAGE_EDITOR, so a better solution would be removing this flag from these properties.

EDIT:
There is also this 🤔

if (vc.name.begins_with("input/") || vc.name.begins_with("import/") || vc.name.begins_with("export/") || vc.name.begins_with("/remap") || vc.name.begins_with("/locale") || vc.name.begins_with("/autoload")) {
vc.flags = PROPERTY_USAGE_STORAGE;
} else {
vc.flags = PROPERTY_USAGE_EDITOR | PROPERTY_USAGE_STORAGE;
}

Definitely need to do a cleanup here.

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