kb(Gantt): add PR review suggestions

Author: IvanDanchev

knowledge-base/gantt-persist-treelist-width-after-refresh.md

@@ -4,7 +4,7 @@ description: How to persist the width of the Gantt's treelist after refreshing i
 type: how-to
 page_title: How to Display Model Fields in the Gantt Tooltip?
 slug: gantt-kb-persist-treelist-width-after-refresh
-tags: gantt, blazor, treelist, width, persist
+tags: gantt, blazor, treelist, width
 res_type: kb
 ticketid: 1690467
 ---
@@ -82,11 +82,10 @@ To persist the current TreeList width, follow the steps below:
 </script>
 
 @code {
-    private TelerikGantt<FlatModel> GanttRef;
+    private TelerikGantt<FlatModel>? GanttRef { get; set; }
     private string ListWidth { get; set; } = "390px";
-
+    private int LastId { get; set; } = 1;
     private List<FlatModel> GanttData { get; set; }
-
     private GanttView SelectedView { get; set; } = GanttView.Week;
 
     private async Task AddRootTask()
@@ -106,7 +105,7 @@ To persist the current TreeList width, follow the steps below:
         var currentListWidth = await JS.InvokeAsync<string>("getListSize");
         ListWidth = currentListWidth;
 
-        GanttRef.Rebind();
+        GanttRef?.Rebind();
     }
 
     private async Task RemoveTask()
@@ -118,25 +117,47 @@ To persist the current TreeList width, follow the steps below:
         var currentListWidth = await JS.InvokeAsync<string>("getListSize");
         ListWidth = currentListWidth;
 
-        GanttRef.Rebind();
+        GanttRef?.Rebind();
     }
 
-    class FlatModel
+    
+    private void UpdateItem(GanttUpdateEventArgs args)
     {
-        public int Id { get; set; }
-        public int? ParentId { get; set; }
-        public string Title { get; set; }
-        public double PercentComplete { get; set; }
-        public DateTime Start { get; set; }
-        public DateTime End { get; set; }
+        var item = args.Item as FlatModel;
+
+        var foundItem = GanttData.FirstOrDefault(i => i.Id.Equals(item.Id));
+
+        if (foundItem != null)
+        {
+            foundItem.Title = item.Title;
+            foundItem.Start = item.Start;
+            foundItem.End = item.End;
+            foundItem.PercentComplete = item.PercentComplete;
+        }
     }
 
-    private int LastId { get; set; } = 1;
+    private void DeleteItem(GanttDeleteEventArgs args)
+    {
+        var item = GanttData.FirstOrDefault(i => i.Id.Equals((args.Item as FlatModel).Id));
+
+        RemoveChildRecursive(item);
+    }
+
+    private void RemoveChildRecursive(FlatModel item)
+    {
+        var children = GanttData.Where(i => item.Id.Equals(i.ParentId)).ToList();
+
+        foreach (var child in children)
+        {
+            RemoveChildRecursive(child);
+        }
+
+        GanttData.Remove(item);
+    }
 
     protected override void OnInitialized()
     {
         GanttData = new List<FlatModel>();
-        var random = new Random();
 
         for (int i = 1; i < 6; i++)
         {
@@ -146,7 +167,7 @@ To persist the current TreeList width, follow the steps below:
                 Title = "Task " + i.ToString(),
                 Start = new DateTime(2021, 7, 5 + i),
                 End = new DateTime(2021, 7, 11 + i),
-                PercentComplete = Math.Round(random.NextDouble(), 2)
+                PercentComplete = Math.Round(Random.Shared.NextDouble(), 2)
             };
 
             GanttData.Add(newItem);
@@ -162,7 +183,7 @@ To persist the current TreeList width, follow the steps below:
                     Title = "    Task " + i + " : " + j.ToString(),
                     Start = new DateTime(2021, 7, 5 + j),
                     End = new DateTime(2021, 7, 6 + i + j),
-                    PercentComplete = Math.Round(random.NextDouble(), 2)
+                    PercentComplete = Math.Round(Random.Shared.NextDouble(), 2)
                 });
 
                 LastId++;
@@ -172,38 +193,14 @@ To persist the current TreeList width, follow the steps below:
         base.OnInitialized();
     }
 
-    private void UpdateItem(GanttUpdateEventArgs args)
-    {
-        var item = args.Item as FlatModel;
-
-        var foundItem = GanttData.FirstOrDefault(i => i.Id.Equals(item.Id));
-
-        if (foundItem != null)
-        {
-            foundItem.Title = item.Title;
-            foundItem.Start = item.Start;
-            foundItem.End = item.End;
-            foundItem.PercentComplete = item.PercentComplete;
-        }
-    }
-
-    private void DeleteItem(GanttDeleteEventArgs args)
-    {
-        var item = GanttData.FirstOrDefault(i => i.Id.Equals((args.Item as FlatModel).Id));
-
-        RemoveChildRecursive(item);
-    }
-
-    private void RemoveChildRecursive(FlatModel item)
+    class FlatModel
     {
-        var children = GanttData.Where(i => item.Id.Equals(i.ParentId)).ToList();
-
-        foreach (var child in children)
-        {
-            RemoveChildRecursive(child);
-        }
-
-        GanttData.Remove(item);
+        public int Id { get; set; }
+        public int? ParentId { get; set; }
+        public string Title { get; set; }
+        public double PercentComplete { get; set; }
+        public DateTime Start { get; set; }
+        public DateTime End { get; set; }
     }
 }
 ````