[#1038] Primary button has colored shadow background

* [#1038] Primary button has colored shadow bg

- Closes #1038
- Also improves share_file_provider_paths content for exporting private data feature

* Reorder parameters

ui-design-lib/src/main/java/co/electriccoin/zcash/ui/design/component/Button.kt

@@ -1,6 +1,5 @@
 package co.electriccoin.zcash.ui.design.component
 
-import android.graphics.BlurMaskFilter
 import androidx.compose.animation.core.animateFloatAsState
 import androidx.compose.animation.core.tween
 import androidx.compose.foundation.border
@@ -74,13 +73,12 @@ fun PrimaryButton(
         modifier = modifier
             .padding(outerPaddingValues)
             .shadow(
-                ZcashTheme.colors.buttonShadowColor,
+                contentColor = textColor,
-                borderRadius = 0.dp,
+                strokeColor = buttonColor,
+                strokeWidth = 1.dp,
                 offsetX = ZcashTheme.dimens.shadowOffsetX,
                 offsetY = ZcashTheme.dimens.shadowOffsetY,
                 spread = ZcashTheme.dimens.shadowSpread,
-                blurRadius = 0.dp,
-                stroke = textColor != MaterialTheme.colorScheme.primary,
             )
             .translationClick(
                 translationX = ZcashTheme.dimens.shadowOffsetX + 6.dp, // + 6dp to exactly cover the bottom shadow
@@ -89,7 +87,7 @@ fun PrimaryButton(
             .defaultMinSize(ZcashTheme.dimens.defaultButtonWidth, ZcashTheme.dimens.defaultButtonHeight)
             .border(1.dp, Color.Black),
         colors = buttonColors(
-            buttonColor,
+            containerColor = buttonColor,
             disabledContainerColor = ZcashTheme.colors.disabledButtonColor,
             disabledContentColor = ZcashTheme.colors.disabledButtonTextColor
         ),
@@ -124,13 +122,11 @@ fun SecondaryButton(
         modifier = modifier
             .padding(outerPaddingValues)
             .shadow(
-                ZcashTheme.colors.buttonShadowColor,
+                contentColor = textColor,
-                borderRadius = 0.dp,
+                strokeColor = textColor,
                 offsetX = ZcashTheme.dimens.shadowOffsetX,
                 offsetY = ZcashTheme.dimens.shadowOffsetY,
                 spread = ZcashTheme.dimens.shadowSpread,
-                blurRadius = 0.dp,
-                stroke = textColor != MaterialTheme.colorScheme.primary,
             )
             .translationClick(
                 translationX = ZcashTheme.dimens.shadowOffsetX + 6.dp, // + 6dp to exactly cover the bottom shadow
@@ -139,7 +135,7 @@ fun SecondaryButton(
             .defaultMinSize(ZcashTheme.dimens.defaultButtonWidth, ZcashTheme.dimens.defaultButtonHeight)
             .border(1.dp, Color.Black),
         colors = buttonColors(
-            buttonColor,
+            containerColor = buttonColor,
             disabledContainerColor = ZcashTheme.colors.disabledButtonColor,
             disabledContentColor = ZcashTheme.colors.disabledButtonTextColor
         ),
@@ -245,23 +241,29 @@ fun DangerousButton(
 
 @Suppress("LongParameterList")
 fun Modifier.shadow(
-    color: Color = Color.Black,
+    contentColor: Color = Color.Black,
-    borderRadius: Dp = 0.dp,
+    strokeColor: Color = Color.Black,
-    blurRadius: Dp = 0.dp,
+    strokeWidth: Dp = 2.dp,
     offsetY: Dp = 0.dp,
     offsetX: Dp = 0.dp,
     spread: Dp = 0f.dp,
-    stroke: Boolean = true,
     modifier: Modifier = Modifier
 ) = this.then(
     modifier.drawBehind {
         this.drawIntoCanvas {
-            val paint = Paint()
+            val strokePaint = Paint()
-            if (stroke) {
+            strokePaint.style = PaintingStyle.Stroke
-                paint.style = PaintingStyle.Stroke
+            strokePaint.strokeWidth = strokeWidth.toPx()
-                paint.strokeWidth = 2f
+
-                paint.color = Color.Black
+            val contentPaint = Paint()
-            }
+            strokePaint.style = PaintingStyle.Fill
+
+            // Reusable inner function to be able to reach modifier and canvas properties
+            fun drawShadowLayer(
+                paint: Paint,
+                color: Color,
+                paddingWidth: Float
+            ) {
                 val frameworkPaint = paint.asFrameworkPaint()
                 val spreadPixel = spread.toPx()
                 val leftPixel = (0f - spreadPixel) + offsetX.toPx()
@@ -269,22 +271,22 @@ fun Modifier.shadow(
                 val rightPixel = (this.size.width + spreadPixel)
                 val bottomPixel = (this.size.height + spreadPixel)
 
-            if (blurRadius != 0.dp) {
-                frameworkPaint.maskFilter =
-                    (BlurMaskFilter(blurRadius.toPx(), BlurMaskFilter.Blur.NORMAL))
-            }
-
                 frameworkPaint.color = color.toArgb()
                 it.drawRoundRect(
-                left = leftPixel,
+                    left = leftPixel + paddingWidth,
-                top = topPixel,
+                    top = topPixel + paddingWidth,
-                right = rightPixel,
+                    right = rightPixel - paddingWidth,
-                bottom = bottomPixel,
+                    bottom = bottomPixel - paddingWidth,
-                radiusX = borderRadius.toPx(),
+                    radiusX = 0f,
-                radiusY = borderRadius.toPx(),
+                    radiusY = 0f,
                     paint
                 )
             }
+
+            // Draw stroke and then content paints
+            drawShadowLayer(strokePaint, strokeColor, 0f)
+            drawShadowLayer(contentPaint, contentColor, strokeWidth.toPx())
+        }
     }
 )
 

ui-design-lib/src/main/java/co/electriccoin/zcash/ui/design/theme/Dimens.kt

@@ -34,7 +34,7 @@ data class Dimens(
     // In screen custom spacings:
     val inScreenZcashLogoHeight: Dp,
     val inScreenZcashLogoWidth: Dp,
-    val inScreenZcashTextLogoHeight: Dp
+    val inScreenZcashTextLogoHeight: Dp,
 )
 
 private val defaultDimens = Dimens(
@@ -54,7 +54,7 @@ private val defaultDimens = Dimens(
     topAppBarZcashLogoHeight = 24.dp,
     inScreenZcashLogoHeight = 100.dp,
     inScreenZcashLogoWidth = 60.dp,
-    inScreenZcashTextLogoHeight = 30.dp
+    inScreenZcashTextLogoHeight = 30.dp,
 )
 
 private val normalDimens = defaultDimens

ui-design-lib/src/main/java/co/electriccoin/zcash/ui/design/theme/internal/Typography.kt

@@ -87,7 +87,7 @@ internal val SecondaryTypography = Typography(
     headlineSmall = TextStyle(
         fontFamily = ArchivoFontFamily,
         fontWeight = FontWeight.Bold,
-        fontSize = 18.sp,
+        fontSize = 20.sp,
         textAlign = TextAlign.Center
     ),
     bodyLarge = TextStyle(

ui-lib/src/main/res/ui/export_data/xml/share_file_provider_paths.xml

@@ -1,6 +1,21 @@
 <?xml version="1.0" encoding="utf-8"?>
 <paths>
     <!-- Android Studio complains about root-path. Search for an alternative way of approaching no_backup folder -->
-    <root-path name="root" path="/data/data/co.electriccoin.zcash.debug/no_backup/co.electricoin.zcash/." />
+    <!-- Another way is to split paths into packages depending on current build type -->
-    <root-path name="root" path="/data/data/co.electriccoin.zcash/no_backup/co.electricoin.zcash/." />
+    <root-path
+        name="root_mainnet_release"
+        path="/data/data/co.electriccoin.zcash/no_backup/co.electricoin.zcash/."
+    />
+    <root-path
+        name="root_mainnet_debug"
+        path="/data/data/co.electriccoin.zcash.debug/no_backup/co.electricoin.zcash/."
+    />
+    <root-path
+        name="root_testnet_release"
+        path="/data/data/co.electriccoin.zcash.testnet/no_backup/co.electricoin.zcash/."
+    />
+    <root-path
+        name="root_testnet_debug"
+        path="/data/data/co.electriccoin.zcash.testnet.debug/no_backup/co.electricoin.zcash/."
+    />
 </paths>

ui-lib/src/main/res/ui/security_warning/values/strings.xml

@@ -13,6 +13,6 @@
     <string name="security_warning_acknowledge">I acknowledge</string>
     <string name="security_warning_back_content_description">Back</string>
     <string name="security_warning_header">Security warning:</string>
-    <string name="security_warning_back">back</string>
+    <string name="security_warning_back">Back</string>
     <string name="security_warning_unable_to_web_browser">Unable to find a web browser app.</string>
 </resources>