You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.
When reconciling a primitive component containing children with a element that doesn't have children then the children are not removed. If the new element has an empty list of children, the children are properly updated.
RobloxRender.updateHostNode at line 272 checks if the children are present in the new element, and if so add/removes children as needed. However, if the node had children but the new element does, it silently ignores the children.
RobloxRender.applyProp does receive the children property being set to nil but at line 112 ignores it but it a comment notes that it should be dealt with elsewhere.
The text was updated successfully, but these errors were encountered:
I suspect that RobloxRender.updateHostNode should call updateVirtualNodeWithChildren with an empty children list when the old element had children but the new element does not.
idiomic
changed the title
RobloxRenderer Does Not Remove All Children
RobloxRenderer doesn't remove all children
Jun 12, 2019
This may be one of those bugs whose functionality is unknowingly being relied upon in current code bases. Submitting a fix and updating current code bases may be dangerous. Would you like me to submit a PR anyways?
Does this still occur in latest master, or is this a duplicate of #209? If it is valid against current master, please provide a repro case, as I'm not quite understanding how this is different from that issue.
When reconciling a primitive component containing children with a element that doesn't have children then the children are not removed. If the new element has an empty list of children, the children are properly updated.
RobloxRender.updateHostNode at line 272 checks if the children are present in the new element, and if so add/removes children as needed. However, if the node had children but the new element does, it silently ignores the children.
RobloxRender.applyProp does receive the children property being set to nil but at line 112 ignores it but it a comment notes that it should be dealt with elsewhere.
The text was updated successfully, but these errors were encountered: