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

[Feature request] ShowNonSerializedField #83

Open
KospY opened this issue Jun 24, 2023 · 1 comment
Open

[Feature request] ShowNonSerializedField #83

KospY opened this issue Jun 24, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@KospY
Copy link

KospY commented Jun 24, 2023

Would be nice to get an attribute to show non serialized fields in inspector.
ODIN and NaughtyAttributes have this option, and it's quite handy for debugging and other specific cases.

Something like:

[ShowNonSerializedField]
private int myInt = 1;

[NonSerialized, ShowNonSerializedField]
public int myInt2 = 10;

Thanks!

@arimger
Copy link
Owner

arimger commented Jun 26, 2023

Hello, thanks for the issue. I agree, this is a really nice feature, but the reason why this is not implemented is quite simple - performance. NaughtyAttributes and Odin (just a guess, don't know the source code) use Reflection to find and process all properties and fields in the active component/type. I wanted to avoid it and keep Toolbox as lightweight as possible. In this case, I can achieve it differently, perhaps with a dedicated attribute to mark complex types. Anyway, I can't promise when this will be implemented.

@arimger arimger added the enhancement New feature or request label Jun 26, 2023
@arimger arimger self-assigned this Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants