Merge pull request #453 from trimble-oss/452-memory-leakage-on-modus-controls

Memory leakage on modus controls #452

Author: amosekr

.github/workflows/ci.yaml

@@ -28,10 +28,10 @@ jobs:
         with:
           persist-credentials: false
 
-      - name: Setup .NET 8
+      - name: Setup .NET 9
         uses: actions/setup-dotnet@v4
         with:
-          dotnet-version: 8.0.x
+          dotnet-version: 9.0.x
           include-prerelease: true
 
       - uses: actions/setup-java@v4
@@ -51,4 +51,4 @@ jobs:
         run: dotnet restore Trimble.Modus.Components.sln
 
       - name: Build MAUI Android
-        run: dotnet build Trimble.Modus.Components.sln -f net8.0-android -c Release --no-restore
+        run: dotnet build Trimble.Modus.Components.sln -f net9.0-android -c Release --no-restore

DemoApp/DemoApp/DemoApp.csproj

@@ -1,8 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
 	<PropertyGroup>
-		<TargetFrameworks>net8.0-android;net8.0-ios;net8.0-maccatalyst</TargetFrameworks>
-		<TargetFrameworks Condition="$([MSBuild]::IsOSPlatform('windows'))">$(TargetFrameworks);net8.0-windows10.0.19041.0</TargetFrameworks>
+		<TargetFrameworks>net9.0-android;net9.0-ios;net9.0-maccatalyst</TargetFrameworks>
+		<TargetFrameworks Condition="$([MSBuild]::IsOSPlatform('windows'))">$(TargetFrameworks);net9.0-windows10.0.19041.0</TargetFrameworks>
 		<!-- Uncomment to also build the tizen app. You will need to install tizen by following this: https://github.com/Samsung/Tizen.NET -->
 		<!-- <TargetFrameworks>$(TargetFrameworks);net7.0-tizen</TargetFrameworks> -->
 		<OutputType>Exe</OutputType>
@@ -14,8 +14,8 @@
 		<!-- Display name -->
 		<ApplicationTitle>DemoApp</ApplicationTitle>
 
-		<!-- App Identifier -->
-		<ApplicationId>com.trimble.ModusMAUI</ApplicationId>
+		<!-- App Identifier -->
+    <ApplicationId>com.trimble.ModusMAUI</ApplicationId>
 		<ApplicationIdGuid>4ca86287-676e-49cb-9a5d-75d22c1b3771</ApplicationIdGuid>
     <PackageVersion>1.0.0</PackageVersion>
 		<!-- Versions -->

DemoApp/DemoApp/Views/ModalSamplePage.xaml.cs

@@ -66,7 +66,7 @@ private void ShowModalClicked(object sender, EventArgs e)
         catch (Exception ex)
         {
             TMToast tMToast = new(ex.Message);
-            tMToast.theme = ToastTheme.Danger;
+            tMToast.theme = ToastTheme.DangerToast;
             tMToast.Show();
         }
     }

DemoApp/DemoApp/Views/ToastSamplePage.xaml

@@ -34,12 +34,12 @@
       <Picker.ItemsSource>
         <x:Array Type="{x:Type theme:ToastTheme}">
           <theme:ToastTheme></theme:ToastTheme>
-          <theme:ToastTheme>Dark</theme:ToastTheme>
-          <theme:ToastTheme>Primary</theme:ToastTheme>
-          <theme:ToastTheme>Secondary</theme:ToastTheme>
-          <theme:ToastTheme>Danger</theme:ToastTheme>
-          <theme:ToastTheme>Warning</theme:ToastTheme>
-          <theme:ToastTheme>Success</theme:ToastTheme>
+          <theme:ToastTheme>DarkToast</theme:ToastTheme>
+          <theme:ToastTheme>PrimaryToast</theme:ToastTheme>
+          <theme:ToastTheme>SecondaryToast</theme:ToastTheme>
+          <theme:ToastTheme>DangerToast</theme:ToastTheme>
+          <theme:ToastTheme>WarningToast</theme:ToastTheme>
+          <theme:ToastTheme>SuccessToast</theme:ToastTheme>
         </x:Array>
       </Picker.ItemsSource>
 

DemoApp/DemoApp/Views/ToastSamplePage.xaml.cs

@@ -9,7 +9,7 @@ public partial class ToastSamplePage : ContentPage
     public ToastSamplePage()
     {
         InitializeComponent();
-        toastPicker.SelectedItem = ToastTheme.Default;
+        toastPicker.SelectedItem = ToastTheme.DefaultToast;
     }
 
     private void OnToastActionTapped(object sender, EventArgs e)

Trimble.Modus.Components/Controls/Button/TMButton.xaml

@@ -9,9 +9,9 @@
              HorizontalOptions="Start">
   <ContentView.Resources>
     <ResourceDictionary>
-      <ResourceDictionary.MergedDictionaries>
+      <!--<ResourceDictionary.MergedDictionaries>
         <local:TMButtonStyles></local:TMButtonStyles>
-      </ResourceDictionary.MergedDictionaries>
+      </ResourceDictionary.MergedDictionaries>-->
       <converters:ImageNullOrEmptyBoolConverter x:Key="ImageNullOrEmptyBoolConverter" />
       <converters:StringNotNullOrEmptyBoolConverter x:Key="StringNotNullOrEmptyBoolConverter" />
     </ResourceDictionary>

Trimble.Modus.Components/Controls/Button/TMButtonStyles.xaml

@@ -5,14 +5,7 @@
                  xmlns:local="clr-namespace:Trimble.Modus.Components"
                  x:Class="Trimble.Modus.Components.DropDownContents"
                  BackgroundColor="{DynamicResource Transparent}"
-                 Title="DropDownPopupPage">
-  <Modus:PopupPage.Resources>
-    <ResourceDictionary>
-      <ResourceDictionary.MergedDictionaries>
-        <local:DropDownStyles />
-      </ResourceDictionary.MergedDictionaries>
-    </ResourceDictionary>
-  </Modus:PopupPage.Resources>
+                 Title="DropDownPopupPage"> 
   <Border x:Name="border"
           VerticalOptions="Start"
           HorizontalOptions="Start"