Replace meta properties with regular properties in Control
#64626
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #61343. Supersedes #61529.
As discussed with @reduz, meta in this case is likely not needed and normal properties should be used. The logic should be overwise unchanged, including ignoring the stored value if it doesn't match the state of the control (e.g. it doesn't matter if the layout mode is stored as "anchors" if it's a child of a container, we just ignore it).
The property is now saved more often, but it shouldn't be a huge bother, as it is a property of the node after all, even if it's not relevant for the runtime.
I'll have to rebase this soon to update for the changes in property revert methods from #64334. But as far as this PR is concerned, this is a cosmetic change. Though some builds will fail, because I didn't update the documentation for temporarily bound methods. They'll soon be gone, so there is no point in doing that.Done