From 1a97a81938f5437446a21fd788a4f1b7fa02f483 Mon Sep 17 00:00:00 2001
From: Sathish-087 <sathish.selvam@syncfusion.com>
Date: Fri, 18 Oct 2024 19:00:56 +0530
Subject: [PATCH] Address review comments

---
 .../GettingStarted/GettingStarted/App.xaml.cs |   2 +-
 .../GettingStarted/AppShell.xaml              |  15 --
 .../GettingStarted/AppShell.xaml.cs           |  10 --
 .../GettingStarted/GettingStarted/App.xaml.cs |   2 +-
 .../GettingStarted/AppShell.xaml              |  15 --
 .../GettingStarted/AppShell.xaml.cs           |  10 --
 .../Behavior/GettingStartedBehavior.cs        | 144 ++++++++++++++++++
 .../GettingStarted/MainPage.xaml              |  63 ++++----
 .../GettingStarted/MainPage.xaml.cs           |  31 +---
 .../GettingStarted/ViewModel/ListViewModel.cs |  24 +++
 .../GettingStarted/GettingStarted/App.xaml.cs |   2 +-
 .../GettingStarted/AppShell.xaml              |  15 --
 .../GettingStarted/AppShell.xaml.cs           |  10 --
 13 files changed, 205 insertions(+), 138 deletions(-)
 delete mode 100644 EffectsView/GettingStarted/GettingStarted/AppShell.xaml
 delete mode 100644 EffectsView/GettingStarted/GettingStarted/AppShell.xaml.cs
 delete mode 100644 NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml
 delete mode 100644 NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml.cs
 create mode 100644 NavigationDrawer/GettingStarted/GettingStarted/Behavior/GettingStartedBehavior.cs
 create mode 100644 NavigationDrawer/GettingStarted/GettingStarted/ViewModel/ListViewModel.cs
 delete mode 100644 SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml
 delete mode 100644 SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml.cs

diff --git a/EffectsView/GettingStarted/GettingStarted/App.xaml.cs b/EffectsView/GettingStarted/GettingStarted/App.xaml.cs
index 9758931..3183fef 100644
--- a/EffectsView/GettingStarted/GettingStarted/App.xaml.cs
+++ b/EffectsView/GettingStarted/GettingStarted/App.xaml.cs
@@ -6,7 +6,7 @@ public App()
         {
             InitializeComponent();
 
-            MainPage = new AppShell();
+            MainPage = new MainPage();
         }
     }
 }
diff --git a/EffectsView/GettingStarted/GettingStarted/AppShell.xaml b/EffectsView/GettingStarted/GettingStarted/AppShell.xaml
deleted file mode 100644
index 8947f53..0000000
--- a/EffectsView/GettingStarted/GettingStarted/AppShell.xaml
+++ /dev/null
@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="UTF-8" ?>
-<Shell
-    x:Class="GettingStarted.AppShell"
-    xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
-    xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
-    xmlns:local="clr-namespace:GettingStarted"
-    Shell.FlyoutBehavior="Disabled"
-    Title="GettingStarted">
-
-    <ShellContent
-        Title="Home"
-        ContentTemplate="{DataTemplate local:MainPage}"
-        Route="MainPage" />
-
-</Shell>
diff --git a/EffectsView/GettingStarted/GettingStarted/AppShell.xaml.cs b/EffectsView/GettingStarted/GettingStarted/AppShell.xaml.cs
deleted file mode 100644
index b9c11f5..0000000
--- a/EffectsView/GettingStarted/GettingStarted/AppShell.xaml.cs
+++ /dev/null
@@ -1,10 +0,0 @@
-namespace GettingStarted
-{
-    public partial class AppShell : Shell
-    {
-        public AppShell()
-        {
-            InitializeComponent();
-        }
-    }
-}
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/App.xaml.cs b/NavigationDrawer/GettingStarted/GettingStarted/App.xaml.cs
index 9758931..3183fef 100644
--- a/NavigationDrawer/GettingStarted/GettingStarted/App.xaml.cs
+++ b/NavigationDrawer/GettingStarted/GettingStarted/App.xaml.cs
@@ -6,7 +6,7 @@ public App()
         {
             InitializeComponent();
 
-            MainPage = new AppShell();
+            MainPage = new MainPage();
         }
     }
 }
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml b/NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml
deleted file mode 100644
index 8947f53..0000000
--- a/NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml
+++ /dev/null
@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="UTF-8" ?>
-<Shell
-    x:Class="GettingStarted.AppShell"
-    xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
-    xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
-    xmlns:local="clr-namespace:GettingStarted"
-    Shell.FlyoutBehavior="Disabled"
-    Title="GettingStarted">
-
-    <ShellContent
-        Title="Home"
-        ContentTemplate="{DataTemplate local:MainPage}"
-        Route="MainPage" />
-
-</Shell>
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml.cs b/NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml.cs
deleted file mode 100644
index b9c11f5..0000000
--- a/NavigationDrawer/GettingStarted/GettingStarted/AppShell.xaml.cs
+++ /dev/null
@@ -1,10 +0,0 @@
-namespace GettingStarted
-{
-    public partial class AppShell : Shell
-    {
-        public AppShell()
-        {
-            InitializeComponent();
-        }
-    }
-}
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/Behavior/GettingStartedBehavior.cs b/NavigationDrawer/GettingStarted/GettingStarted/Behavior/GettingStartedBehavior.cs
new file mode 100644
index 0000000..8301458
--- /dev/null
+++ b/NavigationDrawer/GettingStarted/GettingStarted/Behavior/GettingStartedBehavior.cs
@@ -0,0 +1,144 @@
+namespace GettingStarted
+{
+    using Syncfusion.Maui.Toolkit.NavigationDrawer;
+
+    /// <summary>
+    /// Getting started behavior class.
+    /// </summary>
+    public class GettingStartedBehavior : Behavior<ContentPage>
+    {
+        /// <summary>
+        /// Navigation drawer instance.
+        /// </summary>
+        private SfNavigationDrawer? navigationDrawer;
+
+        /// <summary>
+        /// Image button instance.
+        /// </summary>
+        private ImageButton? imageButton;
+
+        /// <summary>
+        /// List view instance.
+        /// </summary>
+        private ListView? listView;
+
+        /// <summary>
+        /// Label instance.
+        /// </summary>
+        private Label? contentLabel;
+
+        /// <summary>
+        /// Instance of the list view model.
+        /// </summary>
+        private ListViewModel listViewModel = new ListViewModel();
+
+        /// <summary>
+        /// Called when the behavior is attached to a content page.
+        /// </summary>
+        /// <param name="bindable">The content page.</param>
+        protected override void OnAttachedTo(ContentPage bindable)
+        {
+            base.OnAttachedTo(bindable);
+            bindable.Appearing += Bindable_Appearing;
+        }
+
+        /// <summary>
+        /// Handles the appearing event of the content page.
+        /// </summary>
+        /// <param name="sender">The event sender.</param>
+        /// <param name="e">The event arguments.</param>
+        private void Bindable_Appearing(object? sender, EventArgs e)
+        {
+            var page = sender as ContentPage;
+            if (page == null)
+            {
+                return;
+            }
+
+            this.navigationDrawer = page.Content.FindByName<SfNavigationDrawer>("navigationDrawer");
+            this.imageButton = page.Content.FindByName<ImageButton>("hamburgerButton");
+            this.listView = page.Content.FindByName<ListView>("listView");
+            this.contentLabel = page.Content.FindByName<Label>("contentLabel");
+            if (this.imageButton != null)
+            {
+                this.imageButton.Clicked += ImageButton_Clicked;
+            }
+
+            if (this.listView != null)
+            {
+                this.listView.ItemsSource = listViewModel.MenuItems;
+                this.listView.ItemSelected += ListView_ItemSelected;
+            }
+        }
+
+        /// <summary>
+        /// Handles the item selected event of the list view.
+        /// </summary>
+        /// <param name="sender">The event sender.</param>
+        /// <param name="e">The event arguments.</param>
+        private void ListView_ItemSelected(object? sender, SelectedItemChangedEventArgs e)
+        {
+            if (this.navigationDrawer == null || this.contentLabel == null)
+            {
+                return;
+            }
+
+            if (e.SelectedItem.ToString() == "Home")
+            {
+                this.contentLabel.Text = "Home";
+            }
+            else if (e.SelectedItem.ToString() == "Profile")
+            {
+                this.contentLabel.Text = "Profile";
+            }
+            else if (e.SelectedItem.ToString() == "Inbox")
+            {
+                this.contentLabel.Text = "Inbox";
+            }
+            else if (e.SelectedItem.ToString() == "Outbox")
+            {
+                this.contentLabel.Text = "Outbox";
+            }
+            else if (e.SelectedItem.ToString() == "Sent")
+            {
+                this.contentLabel.Text = "Sent";
+            }
+            else if (e.SelectedItem.ToString() == "Draft")
+            {
+                this.contentLabel.Text = "The folder is empty";
+            }
+
+            this.navigationDrawer.ToggleDrawer();
+        }
+
+        /// <summary>
+        /// Handles the clicked event of the hamburger button.
+        /// </summary>
+        /// <param name="sender">The event sender.</param>
+        /// <param name="e">The event arguments.</param>
+        private void ImageButton_Clicked(object? sender, EventArgs e)
+        {
+            this.navigationDrawer?.ToggleDrawer();
+        }
+
+        /// <summary>
+        /// Called when the behavior is detaching from a content page.
+        /// </summary>
+        /// <param name="bindable">The content page.</param>
+        protected override void OnDetachingFrom(ContentPage bindable)
+        {
+            base.OnDetachingFrom(bindable);
+            if (this.imageButton != null)
+            {
+                this.imageButton.Clicked -= ImageButton_Clicked;
+                this.imageButton = null;
+            }
+
+            if (this.listView != null)
+            {
+                this.listView.ItemSelected += ListView_ItemSelected;
+                this.listView = null;
+            }
+        }
+    }
+}
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml b/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml
index b22cc96..96ba8ae 100644
--- a/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml
+++ b/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml
@@ -2,8 +2,13 @@
 <ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              xmlns:navigationdrawer="clr-namespace:Syncfusion.Maui.Toolkit.NavigationDrawer;assembly=Syncfusion.Maui.Toolkit"
+             xmlns:local="clr-namespace:GettingStarted"
              x:Class="GettingStarted.MainPage">
 
+    <ContentPage.Behaviors>
+        <local:GettingStartedBehavior/>
+    </ContentPage.Behaviors>
+
     <navigationdrawer:SfNavigationDrawer x:Name="navigationDrawer">
         <navigationdrawer:SfNavigationDrawer.DrawerSettings>
             <navigationdrawer:DrawerSettings DrawerWidth="250"
@@ -11,22 +16,21 @@
                 <navigationdrawer:DrawerSettings.DrawerHeaderView>
                     <Grid BackgroundColor="#6750A4" RowDefinitions="120,40">
                         <Image Source="user.png"
-                       HeightRequest="110"
-                       Margin="0,10,0,0"
-                       BackgroundColor="#6750A4"
-                       VerticalOptions="Center"
-                       HorizontalOptions="Center"/>
+                               HeightRequest="110"
+                               Margin="0,10,0,0"
+                               BackgroundColor="#6750A4"
+                               VerticalOptions="Center"
+                               HorizontalOptions="Center"/>
                         <Label Text="James Pollock"
-                       Grid.Row="1"
-                       HorizontalTextAlignment="Center"
-                       HorizontalOptions="Center"
-                       FontSize="20"
-                       TextColor="White"/>
+                               Grid.Row="1"
+                               HorizontalTextAlignment="Center"
+                               HorizontalOptions="Center"
+                               FontSize="20"
+                               TextColor="White"/>
                     </Grid>
                 </navigationdrawer:DrawerSettings.DrawerHeaderView>
                 <navigationdrawer:DrawerSettings.DrawerContentView>
-                    <ListView x:Name="listView"
-                      ItemSelected="listView_ItemSelected">
+                    <ListView x:Name="listView">
                         <ListView.ItemTemplate>
                             <DataTemplate>
                                 <ViewCell>
@@ -48,27 +52,26 @@
       BackgroundColor="White" RowDefinitions="Auto,*">
                 <HorizontalStackLayout BackgroundColor="#6750A4" Spacing="10" Padding="5,0,0,0">
                     <ImageButton x:Name="hamburgerButton"
-                         HeightRequest="50"
-                         WidthRequest="50"
-                         HorizontalOptions="Start"
-                         Source="hamburgericon.png"
-                         BackgroundColor="#6750A4"
-                         Clicked="hamburgerButton_Clicked"/>
+                             HeightRequest="50"
+                             WidthRequest="50"
+                             HorizontalOptions="Start"
+                             Source="hamburgericon.png"
+                             BackgroundColor="#6750A4"/>
                     <Label x:Name="headerLabel" 
-               HeightRequest="50" 
-               HorizontalTextAlignment="Center" 
-               VerticalTextAlignment="Center" 
-               Text="Home" FontSize="16" 
-               TextColor="White" 
-               BackgroundColor="#6750A4"/>
+                           HeightRequest="50" 
+                           HorizontalTextAlignment="Center" 
+                           VerticalTextAlignment="Center" 
+                           Text="Home" FontSize="16" 
+                           TextColor="White" 
+                           BackgroundColor="#6750A4"/>
                 </HorizontalStackLayout>
                 <Label Grid.Row="1" 
-          x:Name="contentLabel" 
-          VerticalOptions="Center" 
-          HorizontalOptions="Center" 
-          Text="Content View" 
-          FontSize="14" 
-          TextColor="Black"/>
+                      x:Name="contentLabel" 
+                      VerticalOptions="Center" 
+                      HorizontalOptions="Center" 
+                      Text="Content View" 
+                      FontSize="14" 
+                      TextColor="Black"/>
             </Grid>
         </navigationdrawer:SfNavigationDrawer.ContentView>
     </navigationdrawer:SfNavigationDrawer>
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml.cs b/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml.cs
index 053022c..f11e73e 100644
--- a/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml.cs
+++ b/NavigationDrawer/GettingStarted/GettingStarted/MainPage.xaml.cs
@@ -1,43 +1,14 @@
-using Microsoft.Maui.Platform;
-using Syncfusion.Maui.Toolkit.NavigationDrawer;
-
-namespace GettingStarted
+namespace GettingStarted
 {
     public partial class MainPage : ContentPage
     {
         public MainPage()
         {
             InitializeComponent();
-            List<string> list = new List<string>();
-            list.Add("Home");
-            list.Add("Profile");
-            list.Add("Inbox");
-            list.Add("Out box");
-            list.Add("Sent");
-            list.Add("Draft");
-            listView.ItemsSource = list;
         }
         private void hamburgerButton_Clicked(object sender, EventArgs e)
         {
             navigationDrawer.ToggleDrawer();
         }
-
-        private void listView_ItemSelected(object sender, SelectedItemChangedEventArgs e)
-        {
-            if (e.SelectedItem.ToString() == "Home")
-                contentLabel.Text = "Home";
-            else if (e.SelectedItem.ToString() == "Profile")
-                contentLabel.Text = "Profile";
-            else if (e.SelectedItem.ToString() == "Inbox")
-                contentLabel.Text = "Inbox";
-            else if (e.SelectedItem.ToString() == "Out box")
-                contentLabel.Text = "Out box";
-            else if (e.SelectedItem.ToString() == "Sent")
-                contentLabel.Text = "Sent";
-            else if (e.SelectedItem.ToString() == "Draft")
-                contentLabel.Text = "The folder is empty";
-            navigationDrawer.ToggleDrawer();
-        }
     }
-
 }
diff --git a/NavigationDrawer/GettingStarted/GettingStarted/ViewModel/ListViewModel.cs b/NavigationDrawer/GettingStarted/GettingStarted/ViewModel/ListViewModel.cs
new file mode 100644
index 0000000..141b71a
--- /dev/null
+++ b/NavigationDrawer/GettingStarted/GettingStarted/ViewModel/ListViewModel.cs
@@ -0,0 +1,24 @@
+namespace GettingStarted
+{
+    /// <summary>
+    /// View model class for providing menu item data.
+    /// </summary>
+    public class ListViewModel
+    {
+        /// <summary>
+        /// Gets or sets the collection of menu items.
+        /// </summary>
+        public List<string> MenuItems { get; set; }
+
+        /// <summary>
+        /// Initializes a new instance of the ListViewModel class.
+        /// </summary>
+        public ListViewModel()
+        {
+            MenuItems = new List<string>
+            {
+                "Home", "Profile", "Inbox", "Outbox", "Sent", "Draft"
+            };
+        }
+    }
+}
\ No newline at end of file
diff --git a/SegmentedControl/GettingStarted/GettingStarted/App.xaml.cs b/SegmentedControl/GettingStarted/GettingStarted/App.xaml.cs
index 9758931..3183fef 100644
--- a/SegmentedControl/GettingStarted/GettingStarted/App.xaml.cs
+++ b/SegmentedControl/GettingStarted/GettingStarted/App.xaml.cs
@@ -6,7 +6,7 @@ public App()
         {
             InitializeComponent();
 
-            MainPage = new AppShell();
+            MainPage = new MainPage();
         }
     }
 }
diff --git a/SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml b/SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml
deleted file mode 100644
index 8947f53..0000000
--- a/SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml
+++ /dev/null
@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="UTF-8" ?>
-<Shell
-    x:Class="GettingStarted.AppShell"
-    xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
-    xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
-    xmlns:local="clr-namespace:GettingStarted"
-    Shell.FlyoutBehavior="Disabled"
-    Title="GettingStarted">
-
-    <ShellContent
-        Title="Home"
-        ContentTemplate="{DataTemplate local:MainPage}"
-        Route="MainPage" />
-
-</Shell>
diff --git a/SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml.cs b/SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml.cs
deleted file mode 100644
index b9c11f5..0000000
--- a/SegmentedControl/GettingStarted/GettingStarted/AppShell.xaml.cs
+++ /dev/null
@@ -1,10 +0,0 @@
-namespace GettingStarted
-{
-    public partial class AppShell : Shell
-    {
-        public AppShell()
-        {
-            InitializeComponent();
-        }
-    }
-}