Tomasz Chabora

Results 934 comments of Tomasz Chabora

- [x] IMO the inspector warnings should be folded by default. - [x] Also why are they clickable? https://github.com/user-attachments/assets/a35283b5-f125-449a-83b7-d7c97ccd8ade - [ ] Using unsupported severity results in broken icon: ![image](https://github.com/user-attachments/assets/0c319d62-0d2f-40fa-bc63-5bf7569e498a)...

Went through the changes and overall looks fine. Though I'm not sure if there is need for supporting Strings in info arrays. It's not needed for compatibility, only for convenience.

> I don't know if LocalVector is correct here It's not. LocalVector is for storing data, for passing data around use Vector. > Where should I store the collapse state...

> Maybe it should instead fallback to "warning" severity and print an error? It could fallback to warning, but not sure about the error. It makes sense, but it's being...

The problem with ERR_PRINT_ONCE is that it will appear only once per editor session 🤔 Seems like the property error is also spamming. Maybe a way to fix both is...

I'd make this "system" local to ConfigurationInfo. You just need a HashSet for errors and a method that prints all of them. Then once an error is to appear, add...

https://github.com/godotengine/godot/pull/91124 addresses this.

I think the option name is a bit unclear. "Always show", if you disable it, the shapes are going to show _sometimes_. When exactly? Reversing it into `show_collision_shapes_only_when_selected`, while more...

The shapes are still visible without the gizmo: ![image](https://github.com/user-attachments/assets/39fde5ae-1845-437d-a8b2-cc5f437ddf83) Though it's probably expected? Also you can't select them without a gizmo, but it's probably not a problem.

Related to #95304 and #95140 I think the options were vertical to be more consistent with native dialogs. CC @bruvzg