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

Add a list of all sub-resources used in the scene #75661

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Apr 4, 2023

Closes godotengine/godot-proposals#4567

image

In the proposal I mentioned a dialog, but then realized a popup is enough.

The list shows all sub-resources found in every node in the scene (excluding nested sub-sub-resources). They are grouped by type. If a resource is named, a name is displayed. If not, the owning node's name is displayed instead. Also if a resource is re-used multiple times, number of usages shows too.

This could be probably more fleshed-out, but the original purpose of this feature was to locate built-in scripts in a scene with many of them, and it serves it well.

@KoBeWi KoBeWi added this to the 4.x milestone Apr 4, 2023
@KoBeWi KoBeWi requested a review from a team as a code owner April 4, 2023 21:55
@KoBeWi KoBeWi force-pushed the no_resource_can_hide_from_the_all-seeing_popup branch from 826051e to e9dcf8b Compare April 4, 2023 23:31
Copy link
Member

@fire fire left a comment

Choose a reason for hiding this comment

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

I did not review the code, but from a usability standpoint I was missing this!

@KoBeWi KoBeWi force-pushed the no_resource_can_hide_from_the_all-seeing_popup branch from e9dcf8b to 5a99304 Compare April 5, 2023 19:01
@akien-mga akien-mga merged commit 49a1962 into godotengine:master Apr 5, 2023
@akien-mga
Copy link
Member

Thanks!

@KoBeWi KoBeWi deleted the no_resource_can_hide_from_the_all-seeing_popup branch April 5, 2023 20:54
@YuriSizov YuriSizov modified the milestones: 4.x, 4.1 Apr 11, 2023
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.

Add a list of scene internals
4 participants