Nomination pool/UI fixes (#1105)

* Fix button colors * Fix start staking more info text * Fix validation progress restoration after warning confirmation
novasamatech · Sep 11, 2023 · 0cbac41 · 0cbac41
1 parent 11f47a3
commit 0cbac41
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 8 deletions.
diff --git a/common/src/main/java/io/novafoundation/nova/common/validation/ValidationExecutor.kt b/common/src/main/java/io/novafoundation/nova/common/validation/ValidationExecutor.kt
@@ -59,6 +59,7 @@ class ValidationExecutor : Validatable {
                         payload = payload,
                         autoFixPayload = autoFixPayload,
                         notValidStatus = it,
+                        progressConsumer = progressConsumer,
                         revalidate = { newPayload ->
                             scope.launch {
                                 requireValid(
@@ -102,8 +103,8 @@ class ValidationExecutor : Validatable {
 
     private fun <P, S> createFlowActions(
         payload: P,
-        progressConsumer: ProgressConsumer? = null,
-        autoFixPayload: (original: P, failureStatus: S) -> P = { original, _ -> original },
+        progressConsumer: ProgressConsumer?,
+        autoFixPayload: (original: P, failureStatus: S) -> P,
         notValidStatus: ValidationStatus.NotValid<S>,
         revalidate: (newPayload: P) -> Unit,
         successBlock: (newPayload: P) -> Unit,

diff --git a/common/src/main/res/drawable/primary_chip_background.xml b/common/src/main/res/drawable/primary_chip_background.xml
@@ -10,7 +10,7 @@
 
     <item android:state_enabled="false">
         <shape android:shape="rectangle">
-            <solid android:color="@color/button_background_inactive" />
+            <solid android:color="@color/button_background_inactive_on_gradient" />
 
             <corners android:radius="10dp" />
         </shape>

diff --git a/common/src/main/res/drawable/secondary_chip_background.xml b/common/src/main/res/drawable/secondary_chip_background.xml
@@ -21,7 +21,7 @@
 
     <item android:state_enabled="false">
         <shape android:shape="rectangle">
-            <solid android:color="@color/button_background_inactive" />
+            <solid android:color="@color/button_background_inactive_on_gradient" />
 
             <corners android:radius="10dp" />
         </shape>

diff --git a/common/src/main/res/values/colors.xml b/common/src/main/res/values/colors.xml
@@ -80,7 +80,8 @@
     <color name="button_background_approve">#2FC864</color>
     <color name="button_background_reject">#E53450</color>
     <color name="button_background_inactive_on_gradient">#1A999EC7</color>
-    <color name="button_background_inactive">#1A999EC7</color>
+    <color name="button_background_inactive">#181920</color>
+
     <color name="button_wallet_connect_background">#353D67</color>
     <color name="wallet_connections_background">#101636</color>
 

diff --git a/common/src/main/res/values/strings.xml b/common/src/main/res/values/strings.xml
@@ -139,9 +139,9 @@
     <string name="start_staking_fragment_governance_participation_with_min_amount_accent">participate in governance</string>
 
     <string name="start_staking_fragment_stake_monitoring_monitor_stake">Monitor your stake</string>
-    <string name="start_staking_fragment_stake_monitoring">Rewards &amp; staking status vary over time. %s from time to time</string>
+    <string name="start_staking_fragment_stake_monitoring">Rewards and staking status vary over time. %s from time to time</string>
 
-    <string name="start_staking_fragment_more_info">Find out more information about\nMoonbeam staking over at the %s</string>
+    <string name="start_staking_fragment_more_info">Find out more information about\n%1$s staking over at the %2$s</string>
     <string name="start_staking_fragment_more_info_clicable_part">Nova Wiki</string>
 
     <string name="start_staking_fragment_terms_of_use">See %s</string>

diff --git a/...ture_staking_impl/data/parachainStaking/repository/ParachainStakingConstantsRepository.kt b/...ture_staking_impl/data/parachainStaking/repository/ParachainStakingConstantsRepository.kt
@@ -39,7 +39,7 @@ class RuntimeParachainStakingConstantsRepository(
 
     override suspend fun minimumDelegatorStake(chainId: ChainId): BigInteger {
         return numberConstantOrNull(chainId, "MinDelegatorStk")
-            // Starting from runtime 1500, MinDelegatorStk was removed and only MinDelegation remained
+            // Starting from runtime 2500, MinDelegatorStk was removed and only MinDelegation remained
             ?: minimumDelegation(chainId)
     }
 

diff --git a/...a/feature_staking_impl/presentation/staking/start/landing/StartStakingLandingViewModel.kt b/...a/feature_staking_impl/presentation/staking/start/landing/StartStakingLandingViewModel.kt
@@ -223,6 +223,7 @@ class StartStakingLandingViewModel(
 
         return SpannableFormatter.format(
             resourceManager.getString(R.string.start_staking_fragment_more_info),
+            chain.name,
             clickablePart
         )
     }