[NUI] Make Action id as internal static readonly
authorEunki Hong <eunkiki.hong@samsung.com>
Tue, 25 Jul 2023 16:49:52 +0000 (01:49 +0900)
committertscholb <scholb.kim@samsung.com>
Tue, 1 Aug 2023 07:13:54 +0000 (16:13 +0900)
Since we share the same action id for each Play / Pause / etc,
we don't need to consider overwrite the action id now.

Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>
src/Tizen.NUI/src/public/BaseComponents/AnimatedImageView.cs
src/Tizen.NUI/src/public/BaseComponents/ImageView.cs
src/Tizen.NUI/src/public/BaseComponents/LottieAnimationView.cs
src/Tizen.NUI/src/public/BaseComponents/ViewEnum.cs

index e8ac7c7..13254f5 100755 (executable)
@@ -28,6 +28,10 @@ namespace Tizen.NUI.BaseComponents
     public partial class AnimatedImageView : ImageView
     {
         #region Internal
+        /// <summary>
+        /// Actions property value to Jump to the specified frame.
+        /// </summary>
+        internal static readonly int ActionJumpTo = Interop.AnimatedImageView.AnimatedImageVisualActionJumpToGet();
         #endregion Internal
 
         #region Private
@@ -50,9 +54,6 @@ namespace Tizen.NUI.BaseComponents
         [EditorBrowsable(EditorBrowsableState.Never)]
         public AnimatedImageView() : base()
         {
-            ActionPlay = Interop.AnimatedImageView.AnimatedImageVisualActionPlayGet();
-            ActionPause = Interop.AnimatedImageView.AnimatedImageVisualActionPauseGet();
-            ActionStop = Interop.AnimatedImageView.AnimatedImageVisualActionStopGet();
         }
 
         /// <summary>
@@ -355,13 +356,6 @@ namespace Tizen.NUI.BaseComponents
                 return ret;
             }
         }
-
-        /// <summary>
-        /// Actions property value to Jump to the specified frame.
-        /// This property can be redefined by child class if it use different value.
-        /// </summary>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionJumpTo { get; set; } = Interop.AnimatedImageView.AnimatedImageVisualActionJumpToGet();
         #endregion Property
 
         #region Method
index b552ea5..af61687 100755 (executable)
@@ -891,28 +891,22 @@ namespace Tizen.NUI.BaseComponents
         /// <summary>
         /// Actions property value for Reload image.
         /// </summary>
-        private int ActionReload { get; set; } = Interop.ImageView.ImageVisualActionReloadGet();
+        internal static readonly int ActionReload = Interop.ImageView.ImageVisualActionReloadGet();
 
         /// <summary>
         /// Actions property value to Play animated images.
-        /// This property can be redefined by child class if it use different value.
         /// </summary>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionPlay { get; set; } = Interop.AnimatedImageView.AnimatedImageVisualActionPlayGet();
+        internal static readonly int ActionPlay = Interop.AnimatedImageView.AnimatedImageVisualActionPlayGet();
 
         /// <summary>
         /// Actions property value to Pause animated images.
-        /// This property can be redefined by child class if it use different value.
         /// </summary>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionPause { get; set; } = Interop.AnimatedImageView.AnimatedImageVisualActionPauseGet();
+        internal static readonly int ActionPause = Interop.AnimatedImageView.AnimatedImageVisualActionPauseGet();
 
         /// <summary>
         /// Actions property value to Stop animated images.
-        /// This property can be redefined by child class if it use different value.
         /// </summary>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionStop { get; set; } = Interop.AnimatedImageView.AnimatedImageVisualActionStopGet();
+        internal static readonly int ActionStop = Interop.AnimatedImageView.AnimatedImageVisualActionStopGet();
 
         internal VisualFittingModeType ConvertFittingModetoVisualFittingMode(FittingModeType value)
         {
index 31d8826..f73cc2d 100755 (executable)
@@ -49,10 +49,6 @@ namespace Tizen.NUI.BaseComponents
         /// <since_tizen> 7 </since_tizen>
         public LottieAnimationView(float scale = 1.0f, bool shown = true) : base()
         {
-            ActionPlay = Interop.LottieAnimationView.AnimatedVectorImageVisualActionPlayGet();
-            ActionPause = Interop.LottieAnimationView.AnimatedVectorImageVisualActionPauseGet();
-            ActionStop = Interop.LottieAnimationView.AnimatedVectorImageVisualActionStopGet();
-
             NUILog.Debug($"< constructor GetId={GetId()} >");
             currentStates.url = "";
             currentStates.loopCount = 1;
@@ -442,18 +438,6 @@ namespace Tizen.NUI.BaseComponents
                 return currentStates.redrawInScalingDown;
             }
         }
-
-
-        /// <summary>
-        /// Actions property value to Jump to the specified frame.
-        /// This property can be redefined by child class if it use different value.
-        /// </summary>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionJumpTo { get; set; } = Interop.LottieAnimationView.AnimatedVectorImageVisualActionJumpToGet();
-
-        // This is used for internal purpose. hidden API.
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionSetDynamicProperty { get; set; } = Interop.LottieAnimationView.AnimatedVectorImageVisualActionSetDynamicProperty();
         #endregion Property
 
 
@@ -916,6 +900,14 @@ namespace Tizen.NUI.BaseComponents
 
 
         #region Internal
+        /// <summary>
+        /// Actions property value to Jump to the specified frame.
+        /// </summary>
+        internal static readonly int ActionJumpTo = Interop.LottieAnimationView.AnimatedVectorImageVisualActionJumpToGet();
+
+        // This is used for internal purpose.
+        internal static readonly int ActionSetDynamicProperty = Interop.LottieAnimationView.AnimatedVectorImageVisualActionSetDynamicProperty();
+
         internal class VisualEventSignalArgs : EventArgs
         {
             public int VisualIndex
index 825e27d..639bd54 100755 (executable)
@@ -166,10 +166,8 @@ namespace Tizen.NUI.BaseComponents
         /// <summary>
         /// Actions property value to update visual property.
         /// Note : Only few kind of properies can be update. Update with invalid property action is undefined.
-        /// This property can be redefined by child class if it use different value.
         /// </summary>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected int ActionUpdateProperty { get; set; } = Interop.Visual.GetActionUpdateProperty();
+        internal static readonly int ActionUpdateProperty = Interop.Visual.GetActionUpdateProperty();
 
         internal enum PropertyRange
         {