Merge pull request #133 from UnityTech/textselection

kgdev · web-flow · commit 6438e0b91a51 · 2019-04-19T15:06:29.000+08:00
fix assertion triggered by disposing a pageView before its initializa…
diff --git a/Runtime/widgets/binding.cs b/Runtime/widgets/binding.cs
@@ -143,6 +143,8 @@ public void detachRootWidget() {
                 return;
             }
             
+            //The former widget tree must be layout first before its destruction
+            this.drawFrame();
             this.attachRootWidget(null);
             this.buildOwner.buildScope(this._renderViewElement);
             this.buildOwner.finalizeTree();

Original file line number	Diff line number	Diff line change
`@@ -143,6 +143,8 @@ public void detachRootWidget() {`
`143`	`143`	`return;`
`144`	`144`	`}`
`145`	`145`
	`146`	`+ //The former widget tree must be layout first before its destruction`
	`147`	`+ this.drawFrame();`
`146`	`148`	`this.attachRootWidget(null);`
`147`	`149`	`this.buildOwner.buildScope(this._renderViewElement);`
`148`	`150`	`this.buildOwner.finalizeTree();`