-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Open
Labels
area-controls-generalGeneral issues that span multiple controls, or common base classes such as View or ElementGeneral issues that span multiple controls, or common base classes such as View or Elementproposal/underconsiderations/triagedIssue has been reviewedIssue has been reviewedt/bugSomething isn't workingSomething isn't working
Milestone
Description
Description
Currently with layouts the flow of operations will go (Similar steps for OnApp)
- Remove view
- Removed from logical view
- OnChildRemoved is called
maui/src/Controls/src/Core/Layout/Layout.cs
Lines 269 to 286 in 776dad9
public void RemoveAt(int index) { if (index >= Count) { return; } var child = _children[index]; _children.RemoveAt(index); if (child is Element element) { RemoveLogicalChild(element); } OnRemove(index, child); } - xplat events fire
- handler processes remove
I think those last two steps should be flipped.
Once our code has finished processing the remove then we should fire all the eventing paths that are going on inside of OnChildRemoved
Related work here
#23408
Metadata
Metadata
Assignees
Labels
area-controls-generalGeneral issues that span multiple controls, or common base classes such as View or ElementGeneral issues that span multiple controls, or common base classes such as View or Elementproposal/underconsiderations/triagedIssue has been reviewedIssue has been reviewedt/bugSomething isn't workingSomething isn't working