Closed jgersti closed 2 years ago
See some of my comments on #176 and #175 for thoughts on expanding the PR to support those properties. Adding to this PR or doing as a separate PR both seem fine.
Do you want to merge as is or keep iterating on the PR?
merge it. the other stuff needs some more thoughts (and time)
Happy to merge but curious what you think on adding attach/detach support..wouldn't be difficult.
AddBooleanHandler(Property.CONNECTED, b => b.IsAttached, (b,v) => {if(v) b.Attach(); else b.Detach(); });
Adds attached as a readonly property. (#174)
We should take a look what other properties can be easily incorporated in this PR.