Added loaded event usage to Marquee to reduce events while unloaded and not disposed

Author: Avid29

components/Marquee/src/Marquee.Events.cs

@@ -23,9 +23,29 @@ public partial class Marquee
     /// Event raised when the Marquee completes scrolling.
     /// </summary>
     public event EventHandler? MarqueeCompleted;
+    
+    private void Marquee_Loaded(object sender, RoutedEventArgs e)
+    {
+        // While loaded, detach the loaded event and attach the unloaded event
+        this.Loaded -= this.Marquee_Loaded;
+        this.Unloaded += Marquee_Unloaded;
+
+        // Attach other events
+        if (_marqueeContainer is not null)
+        {
+            _marqueeContainer.SizeChanged += Container_SizeChanged;
+        }
+
+        if (_marqueeStoryboard is not null)
+        {
+            _marqueeStoryboard.Completed += StoryBoard_Completed;
+        }
+    }
 
     private void Marquee_Unloaded(object sender, RoutedEventArgs e)
     {
+        // Restore the loaded event and detach the unloaded event 
+        this.Loaded += Marquee_Loaded;
         this.Unloaded -= Marquee_Unloaded;
 
         if (_marqueeContainer is not null)

components/Marquee/src/Marquee.cs

@@ -73,9 +73,9 @@ protected override void OnApplyTemplate()
         _marqueeContainer.SizeChanged += Container_SizeChanged;
 
         // Swapping tabs in TabView caused errors where the control would unload and never reattach events.
-        // Hotfix: Don't detach events. This should be fine because the GC will handle it.
-        // However, more research is required.
-        //Unloaded += Marquee_Unloaded;
+        // Hotfix: Track the loaded event. This should be fine because the GC will handle detaching the Loaded
+        // event on disposal. However, more research is required
+        Loaded += this.Marquee_Loaded;
 
         VisualStateManager.GoToState(this, GetVisualStateName(Direction), false);
         VisualStateManager.GoToState(this, GetVisualStateName(Behavior), false);