-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Cleaning up empty nested classes #3998
Comments
Hmm, this is supposed to work. The clearing of "empty" class values is done in tiled/src/libtiled/properties.cpp Lines 90 to 96 in ee52b20
|
Certainly.
|
It looks like the scheme works but only if the nesting depth is greater than one. So the property itself with depth 1 is not cleared, regardless of whether the class is assigned to the object on the map or not. If it is not assigned it makes sense, but if it is assigned then it is redundant to have an empty one. |
Nested class properties remain even if they do not have a single subproperty. They are visible in the interface and then saved in the json map. They appear if you clear all nested properties manually but forget to delete the nested class itself. They also remain after
setProperty(path, undefined)
is executed.It would be convenient if they were cleaned automatically.
The text was updated successfully, but these errors were encountered: