Merge pull request #57 from RS7-TizenReferenceSampleApplication/nicesj.park

Set focus on the layout when it is done to build.

1 files changed, 30 insertions, 5 deletions

diff --git a/NewPlayer/NewPlayer/Views/LibraryPage.xaml.cs b/NewPlayer/NewPlayer/Views/LibraryPage.xaml.cs
index fdf649c..5a3761a 100644
--- a/NewPlayer/NewPlayer/Views/LibraryPage.xaml.cs
+++ b/NewPlayer/NewPlayer/Views/LibraryPage.xaml.cs
@@ -28,11 +28,6 @@ namespace NewPlayer.Views
                 GridView.Children.Add(new ItemView(item), index / 3, index % 3);
                 index++;
             }
-
-            if (index > 0)
-            {
-                GridView.Children[0].FindByName<Button>("FocusButton")?.Focus();
-            }
         }
 
         private void OnPropertyChanged(object sender, System.ComponentModel.PropertyChangedEventArgs e)
@@ -42,10 +37,40 @@ namespace NewPlayer.Views
                 UpdateGridItems();
             }
         }
+
+        protected override void OnAppearing()
+        {
+            base.OnAppearing();
+
+            GridView.LayoutChanged += OnGridViewLayoutChanged;
+        }
+
+        private void OnGridViewLayoutChanged(object sender, System.EventArgs e)
+        {
+            if (GridView.Children != null && GridView.Children.Count > 0)
+            {
+                var item = GridView.Children[0];
+                if (item != null)
+                {
+                    item.FindByName<Button>("FocusButton")?.Focus();
+                }
+            }
+        }
+
         public LibraryPage()
         {
             InitializeComponent();
+
             PropertyChanged += OnPropertyChanged;
+
+            // In a case of Tizen, the Items property is ready before initializing the LibraryView.
+            // Therefore, the property changed event can be skipped.
+            // To prevent from missing the changed event, we should check the Items first.
+            // If it is ready, we can update the Grid.
+            if (Items != null)
+            {
+                UpdateGridItems();
+            }
         }
     }
 }