Fixed Border Radius Alignment

[Problem] [TNINE-4268] When scale factor is default, the border radious is not aligned with window content.

[Cause & Measure]
 Cause   : CornerRadius was not calucalted properly
 Measure : Used proper calculation for BorderRadius

Change-Id: If67aea9e04febb2139a4b32a43280aa5d7bfad54
Signed-off-by: Mobaswirul Islam/NC eXperience Group /SRBD/Engineer/Samsung Electronics <mobaswirul.i@samsung.com>

diff --git a/Notifications/Common/AppConstants.cs b/Notifications/Common/AppConstants.cs
index 618eb3341a87fb7acdcb07d1a67ba169b370ebc4..c9bdc2bdfc35c478618d665d05cb346edf5e1642 100644 (file)
--- a/Notifications/Common/AppConstants.cs
+++ b/Notifications/Common/AppConstants.cs
@@ -28,13 +28,16 @@ namespace Notifications.Common
 
         public static Position2D DefaultWindowPosition = new Position2D(480, 170);
 
-        public const int BorderHeight = 52;
         public const int HeaderHeight = 64;
         public const int NotificationItemHeight = 80;
-        public const int BorderCornerRadius = 24;
-        public const int BaseViewCornerRadius = 24;
         public const int TextPixelSize = 24;
 
+        public const int BorderHeight = 48;
+        public const int BorderThickness = 6;
+        public const int BorderCornerRadius = 26;
+        public const int BorderWindowPadding = 6;
+
+        public const int BaseViewCornerRadius = 26;
         public static Extents BaseViewPadding = new Extents(0, 0, 20, 20);
         public static Extents HeaderPadding = new Extents(16, 16, 8, 8);
         public static Extents DetailContentMargin = new Extents(64, 64, 8, 8);

diff --git a/Notifications/CustomBorder.cs b/Notifications/CustomBorder.cs
index 69290d0696ae692800abf92652296270fd84a167..ef8ec6def258145aec29519a637cfa7e0e187ca3 100644 (file)
--- a/Notifications/CustomBorder.cs
+++ b/Notifications/CustomBorder.cs
@@ -29,9 +29,10 @@ namespace Notifications
         private View borderView;
 
         private static Size2D IconSize = new Size2D(48, 48);
-        private const int BorderThickness = 4;
-        private const int BottomViewHeight = 48;
-        private const int BorderCornerRadius = 24;
+        private const int BorderThickness = AppConstants.BorderThickness;
+        private const int BottomViewHeight = AppConstants.BorderHeight;
+        private const int BorderCornerRadius = AppConstants.BorderCornerRadius;
+        private const int BorderWindowPadding = AppConstants.BorderWindowPadding;
 
         public CustomBorder() : base()
         {

diff --git a/Notifications/Views/BaseView.cs b/Notifications/Views/BaseView.cs
index 6cd0d18e7f304fc5bad07215fd8375f8aff3c65c..4b189492050036edffa9008228b8ceaddc9f027a 100644 (file)
--- a/Notifications/Views/BaseView.cs
+++ b/Notifications/Views/BaseView.cs
@@ -50,14 +50,15 @@ namespace Notifications.Views
         {
             ThemeChangeSensitive = true;
             Size2D = Window.Instance.Size;
-            CornerRadius = AppConstants.BaseViewCornerRadius;
+            CornerRadius = (AppConstants.BorderCornerRadius - AppConstants.BorderWindowPadding).SpToPx();
             Layout = new LinearLayout()
             {
                 LinearOrientation = LinearLayout.Orientation.Vertical,
                 HorizontalAlignment = HorizontalAlignment.Begin,
                 VerticalAlignment = VerticalAlignment.Top,
-                Padding = AppConstants.BaseViewPadding,
+                Padding = AppConstants.BorderWindowPadding,
             };
+
             AddTopView();
             UpdateContent();
             this.SetBinding(IsContentAvailableProperty, "IsNotificationsPresent");

diff --git a/packaging/org.tizen.notifications-1.0.2.tpk b/packaging/org.tizen.notifications-1.0.2.tpk
index a8b527381b978f04bbc7cbc856d2fe3987ae1c51..00f84cbb663a2bf59bf839b112df5776e657d3f5 100644 (file)
Binary files a/packaging/org.tizen.notifications-1.0.2.tpk and b/packaging/org.tizen.notifications-1.0.2.tpk differ