Add #region to specify Linker Work-Around (#920)

* Add #region to specify Linker work-around

* Update Xamarin.CommunityToolkit.csproj

Co-authored-by: Pedro Jesus <[email protected]>
Co-authored-by: Gerald Versluis <[email protected]>
Co-authored-by: Gerald Versluis <[email protected]>

Author: 3 people

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/IconTintColor/IconTintColorEffectRouter.shared.cs

@@ -7,13 +7,15 @@ public class IconTintColorEffectRouter : RoutingEffect
 		public IconTintColorEffectRouter()
 			: base(EffectIds.IconTintColor)
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation
 #if __ANDROID__
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.Android.Effects.IconTintColorEffectRouter();
 #elif __IOS__
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.iOS.Effects.IconTintColorEffectRouter();
 #endif
+			#endregion
 		}
 	}
 }

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/RemoveBorder/RemoveBorderEffect.shared.cs

@@ -8,13 +8,16 @@ public class RemoveBorderEffect : RoutingEffect
 		public RemoveBorderEffect()
 			: base(EffectIds.RemoveBorder)
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation
+
 #if __IOS__
 			if (DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.iOS.Effects.RemoveBorderEffect();
 #elif __ANDROID__
 			if (DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.Android.Effects.RemoveBorderEffect();
 #endif
+			#endregion
 		}
 	}
 }

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/SafeArea/SafeAreaEffectRouter.shared.cs

@@ -8,10 +8,12 @@ public class SafeAreaEffectRouter : RoutingEffect
 		public SafeAreaEffectRouter()
 			: base(EffectIds.SafeArea)
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation
 #if __IOS__
 			if (DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.iOS.Effects.SafeAreaEffectRouter();
 #endif
+			#endregion
 		}
 	}
 }

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/SelectAllText/SelectAllTextEffect.shared.cs

@@ -8,13 +8,15 @@ public class SelectAllTextEffect : RoutingEffect
 		public SelectAllTextEffect()
 			: base(EffectIds.SelectAllText)
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation 
 #if __IOS__
 			if (DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.iOS.Effects.SelectAllTextEffect();
 #elif __ANDROID__
 			if (DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.Android.Effects.SelectAllTextEffect();
 #endif
+			#endregion
 		}
 	}
 }

src/CommunityToolkit/Xamarin.CommunityToolkit/Effects/Touch/TouchEffect.shared.cs

@@ -467,6 +467,7 @@ public event EventHandler<TouchCompletedEventArgs> Completed
 		public TouchEffect()
 			: base(EffectIds.TouchEffect)
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation
 #if __ANDROID__
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.Android.Effects.PlatformTouchEffect();
@@ -483,6 +484,7 @@ public TouchEffect()
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.UWP.Effects.PlatformTouchEffect();
 #endif
+			#endregion
 		}
 
 		public static bool GetIsAvailable(BindableObject bindable)

src/CommunityToolkit/Xamarin.CommunityToolkit/Extensions/TranslateExtension.shared.cs

@@ -17,9 +17,10 @@ public class TranslateExtension : IMarkupExtension<BindingBase>
 		public BindingBase ProvideValue(IServiceProvider serviceProvider)
 		{
 #if !NETSTANDARD1_0
-
+			#region Required work-around to prevent linker from removing the implementation
 			if (DateTime.Now.Ticks < 0)
 				_ = LocalizationResourceManager.Current[Text];
+			#endregion
 
 			var binding = new Binding
 			{

src/CommunityToolkit/Xamarin.CommunityToolkit/Views/RangeSlider/ThumbFrame.shared.cs

@@ -6,10 +6,12 @@ sealed class ThumbFrame : Frame
 	{
 		public ThumbFrame()
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation
 #if __ANDROID__
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new ThumbFrameRenderer(null);
 #endif
+			#endregion
 		}
 	}
 }

src/CommunityToolkit/Xamarin.CommunityToolkit/Views/SideMenuView/SideMenuView.shared.cs

@@ -89,13 +89,15 @@ public static readonly BindableProperty MenuGestureEnabledProperty
 
 		public SideMenuView()
 		{
+			#region Required work-around to prevent linker from removing the platform-specific implementation
 #if __ANDROID__
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.Android.UI.Views.SideMenuViewRenderer(null);
 #elif __IOS__
 			if (System.DateTime.Now.Ticks < 0)
 				_ = new Xamarin.CommunityToolkit.iOS.UI.Views.SideMenuViewRenderer();
 #endif
+			#endregion
 		}
 
 		public new ISideMenuList<View> Children

src/CommunityToolkit/Xamarin.CommunityToolkit/Xamarin.CommunityToolkit.csproj

@@ -31,6 +31,7 @@
     <DebugType>portable</DebugType>
     <Configurations>Debug;Release</Configurations>
     <EnableDefaultCompileItems>false</EnableDefaultCompileItems>
+    <NoWarn>SA1123</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)'=='Debug' ">
     <!-- Manage TargetFrameworks for development (Debug Mode) -->