Merge pull request #57 from unoplatform/dev/xygu/50/defer-load-strat-lazy-nre

XiaotianNetlift · web-flow · commit b7cfc2dc7f4f · 2019-09-18T11:11:46.000-04:00
fixed NRE when using search-box
diff --git a/Microsoft.Toolkit.Uwp.SampleApp.Shared/Shell.SamplePicker.cs b/Microsoft.Toolkit.Uwp.SampleApp.Shared/Shell.SamplePicker.cs
@@ -67,6 +67,9 @@ private void HideSamplePicker()
 
         private async void ShowSamplePicker(Sample[] samples = null, bool group = false)
         {
+            // force materialization
+            FindName("SamplePickerGrid");
+
             if (samples == null && _currentSample != null)
             {
                 var category = await Samples.GetCategoryBySample(_currentSample);
@@ -127,6 +130,7 @@ private void NavView_ItemInvoked(Microsoft.UI.Xaml.Controls.NavigationView sende
         private void NavView_ItemInvoked(Windows.UI.Xaml.Controls.NavigationView sender, Windows.UI.Xaml.Controls.NavigationViewItemInvokedEventArgs args)
 #endif
         {
+            // force materialization
             FindName("SamplePickerGrid");
 
             if (args.InvokedItem is SampleCategory category)

Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,9 @@ private void HideSamplePicker()`
`67`	`67`
`68`	`68`	`private async void ShowSamplePicker(Sample[] samples = null, bool group = false)`
`69`	`69`	`{`
	`70`	`+ // force materialization`
	`71`	`+ FindName("SamplePickerGrid");`
	`72`	`+`
`70`	`73`	`if (samples == null && _currentSample != null)`
`71`	`74`	`{`
`72`	`75`	`var category = await Samples.GetCategoryBySample(_currentSample);`
`@@ -127,6 +130,7 @@ private void NavView_ItemInvoked(Microsoft.UI.Xaml.Controls.NavigationView sende`
`127`	`130`	`private void NavView_ItemInvoked(Windows.UI.Xaml.Controls.NavigationView sender, Windows.UI.Xaml.Controls.NavigationViewItemInvokedEventArgs args)`
`128`	`131`	`#endif`
`129`	`132`	`{`
	`133`	`+ // force materialization`
`130`	`134`	`FindName("SamplePickerGrid");`
`131`	`135`
`132`	`136`	`if (args.InvokedItem is SampleCategory category)`