[NUI] introduce CreateViewStyle that is alternative to GetViewStyle (#1681)
authorYeongJong Lee <cleanlyj@naver.com>
Wed, 10 Jun 2020 08:08:39 +0000 (17:08 +0900)
committerGitHub <noreply@github.com>
Wed, 10 Jun 2020 08:08:39 +0000 (17:08 +0900)
`CreateViewStyle` is alternative method of `GetViewStyle`.
To resolve CA1721, please use `CreateViewStyle` instead.
```
warning CA1721: The property name 'ViewStyle' is confusing given the existence of
method 'GetViewStyle'. Rename or remove one of these members.
```

`GetViewStyle` should be deprecated(or removed) if there is no
compatibility issue.

20 files changed:
src/Tizen.NUI.Components/Controls/Button.cs
src/Tizen.NUI.Components/Controls/Control.cs
src/Tizen.NUI.Components/Controls/DropDown.cs
src/Tizen.NUI.Components/Controls/FlexibleView/FlexibleView.cs
src/Tizen.NUI.Components/Controls/ImageScrollBar.cs
src/Tizen.NUI.Components/Controls/Loading.cs
src/Tizen.NUI.Components/Controls/Pagination.cs
src/Tizen.NUI.Components/Controls/Popup.cs
src/Tizen.NUI.Components/Controls/Progress.cs
src/Tizen.NUI.Components/Controls/Scrollbar.cs
src/Tizen.NUI.Components/Controls/Slider.cs
src/Tizen.NUI.Components/Controls/Switch.cs
src/Tizen.NUI.Components/Controls/Tab.cs
src/Tizen.NUI.Components/Controls/Toast.cs
src/Tizen.NUI.Wearable/src/public/CircularPagination.cs
src/Tizen.NUI.Wearable/src/public/CircularProgress.cs
src/Tizen.NUI.Wearable/src/public/CircularScrollbar.cs
src/Tizen.NUI.Wearable/src/public/CircularSlider.cs
src/Tizen.NUI.Wearable/src/public/Popup.cs
src/Tizen.NUI/src/public/BaseComponents/View.cs

index f52cf41..3d8d160 100755 (executable)
@@ -936,7 +936,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default button style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new ButtonStyle();
         }
index e75413a..0a46c99 100755 (executable)
@@ -260,7 +260,7 @@ namespace Tizen.NUI.Components
 
         /// This will be public opened in tizen_5.5 after ACR done. Before ACR, need to be hidden as inhouse API.
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new ControlStyle();
         }
index 02768e6..6fc1eba 100755 (executable)
@@ -615,7 +615,7 @@ namespace Tizen.NUI.Components
         /// <since_tizen> 6 </since_tizen>
         /// This will be public opened in tizen_5.5 after ACR done. Before ACR, need to be hidden as inhouse API.
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new DropDownStyle();
         }
@@ -1424,7 +1424,7 @@ namespace Tizen.NUI.Components
             /// <returns>The empty.</returns>
             /// This will be public opened in tizen_5.5 after ACR done. Before ACR, need to be hidden as inhouse API.
             [EditorBrowsable(EditorBrowsableState.Never)]
-            protected override ViewStyle GetViewStyle()
+            protected override ViewStyle CreateViewStyle()
             {
                 return new DropDownItemStyle();
             }
index 22bbcb0..2a9a3c5 100755 (executable)
@@ -472,7 +472,7 @@ namespace Tizen.NUI.Components
         /// <since_tizen> 6 </since_tizen>
         /// This will be public opened in tizen_6.0 after ACR done. Before ACR, need to be hidden as inhouse API.
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return null;
         }
index 16f66db..43ce43c 100755 (executable)
@@ -460,7 +460,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default scrollbar style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new ScrollBarStyle();
         }
index 380379e..8924dd7 100755 (executable)
@@ -195,7 +195,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default loading style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new LoadingStyle();
         }
index 0854591..a403af8 100755 (executable)
@@ -274,7 +274,7 @@ namespace Tizen.NUI.Components
         /// <since_tizen> 6 </since_tizen>
         /// This will be public opened in tizen_5.5 after ACR done. Before ACR, need to be hidden as inhouse API.
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new PaginationStyle();
         }
index 629dd21..9057961 100755 (executable)
@@ -724,7 +724,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default popup style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new PopupStyle();
         }
index 7bcce34..4dbf591 100755 (executable)
@@ -483,7 +483,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default progress style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new ProgressStyle();
         }
index 7e02aa2..eeb37a4 100644 (file)
@@ -374,7 +374,7 @@ namespace Tizen.NUI.Components
 
         /// <inheritdoc/>
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new ScrollbarStyle();
         }
index 53d835b..74d605b 100755 (executable)
@@ -763,7 +763,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default slider style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new SliderStyle();
         }
index e9d7408..da159ae 100755 (executable)
@@ -341,7 +341,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default switch style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new SwitchStyle();
         }
index 5d50844..d306d30 100755 (executable)
@@ -455,7 +455,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default tab style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new TabStyle();
         }
index 14d90a3..d357239 100755 (executable)
@@ -337,7 +337,7 @@ namespace Tizen.NUI.Components
         /// </summary>
         /// <returns>The default toast style.</returns>
         /// <since_tizen> 8 </since_tizen>
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new ToastStyle();
         }
index 1691f92..607518a 100755 (executable)
@@ -556,7 +556,7 @@ namespace Tizen.NUI.Wearable
         /// <since_tizen> 8 </since_tizen>
         /// This will be public opened in tizen_6.0 after ACR done. Before ACR, need to be hidden as inhouse API.
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new CircularPaginationStyle();
         }
index 3ce7821..3d2e46b 100755 (executable)
@@ -402,7 +402,7 @@ namespace Tizen.NUI.Wearable
         /// </summary>
         /// <returns>The default progress style.</returns>
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new CircularProgressStyle();
         }
index df7d50d..1155033 100644 (file)
@@ -356,7 +356,7 @@ namespace Tizen.NUI.Wearable
 
         /// <inheritdoc/>
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new CircularScrollbarStyle();
         }
index 4bb425b..ca8d7de 100755 (executable)
@@ -489,7 +489,7 @@ namespace Tizen.NUI.Wearable
         /// </summary>
         /// <returns>The default progress style.</returns>
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new CircularSliderStyle();
         }
index 6f84efd..98f52ba 100755 (executable)
@@ -430,7 +430,7 @@ namespace Tizen.NUI.Wearable
         /// </summary>
         /// <returns>The default popup style.</returns>
         [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override ViewStyle GetViewStyle()
+        protected override ViewStyle CreateViewStyle()
         {
             return new PopupStyle();
         }
index 7e97380..50f69f7 100755 (executable)
@@ -2307,9 +2307,19 @@ namespace Tizen.NUI.BaseComponents
         /// </summary>
         /// <since_tizen> 6 </since_tizen>
         /// This will be public opened in tizen_6.0 after ACR done. Before ACR, need to be hidden as inhouse API.
+        // TODO: It should be deprecated. please use CreateViewStyle instead.
         [EditorBrowsable(EditorBrowsableState.Never)]
         protected virtual ViewStyle GetViewStyle()
         {
+            return CreateViewStyle();
+        }
+
+        /// <summary>
+        /// Create Style, it is abstract function and must be override.
+        /// </summary>
+        [EditorBrowsable(EditorBrowsableState.Never)]
+        protected virtual ViewStyle CreateViewStyle()
+        {
             return new ViewStyle();
         }