[Editor] Generate and display documentation for the properties generated by PropertyListHelper.#115253
Conversation
b0c2c5b to
1e589f1
Compare
There was a problem hiding this comment.
I actually made proposal for this, not sure if you had seen it: godotengine/godot-proposals#13965
In my proposal I suggested having a separate list for dynamic properties, since they are a group. So instead of item_{index}/text you'd have something like
Property Group Items:
Prefix: item_
Count: item_count
bool disabled
description
String text
description
etc.
(somewhat similar to how enums are organized)
The Count information would also allow to get rid of Note: in every description. However it's rather significant change over the current implementation (it'd also affect XML structure), so probably up to discussion.
The implementation itself looks good, aside from missing Open Documentation button in the context menu for dynamic properties. Also needs rebase to include recently added Curve helpers.
1e589f1 to
68b6b25
Compare
Not sure if its worth adding new list for the |
…ted by `PropertyListHelper`.
68b6b25 to
2c2ed8d
Compare
KoBeWi
left a comment
There was a problem hiding this comment.
Not a fan of having a note in every description, but the implementation itself is fine. It can be improved upon later if needed.
AThousandShips
left a comment
There was a problem hiding this comment.
Approving the new documentation entries
|
Thanks! |
Adds documentation for the dynamic properties generated by
PropertyListHelper(e.g, menu items).