Use correct icon tint in voice message preview player (#1672)

Marco Romano · web-flow · commit f3c4c6218e1c · 2023-10-27T16:24:16.000Z
Action item from design review.
diff --git a/libraries/textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/components/VoiceMessagePreview.kt b/libraries/textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/components/VoiceMessagePreview.kt
@@ -129,12 +129,14 @@ private fun PlayerButton(
 private fun PauseIcon() = Icon(
     resourceId = R.drawable.ic_pause,
     contentDescription = stringResource(id = CommonStrings.a11y_pause),
+    tint = ElementTheme.colors.iconSecondary,
 )
 
 @Composable
 private fun PlayIcon() = Icon(
     resourceId = R.drawable.ic_play,
     contentDescription = stringResource(id = CommonStrings.a11y_play),
+    tint = ElementTheme.colors.iconSecondary,
 )
 
 @PreviewsDayNight
diff --git a/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer.components_null_VoiceMessagePreview-D-16_16_null,NEXUS_5,1.0,en].png b/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer.components_null_VoiceMessagePreview-D-16_16_null,NEXUS_5,1.0,en].png
@@ -1,3 +1,3 @@
 version https://git-lfs.github.com/spec/v1
-oid sha256:ff4763716e5283a13f7028532cf22953f08d93512bd19ac98e774fd950173ad5
-size 22534
+oid sha256:a53da79f9919d7f55884273fdcc473443a2d2a8eac36bc1e7c79baecff40b69e
+size 22526
diff --git a/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer.components_null_VoiceMessagePreview-N-16_17_null,NEXUS_5,1.0,en].png b/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer.components_null_VoiceMessagePreview-N-16_17_null,NEXUS_5,1.0,en].png
@@ -1,3 +1,3 @@
 version https://git-lfs.github.com/spec/v1
-oid sha256:a88306846e0bf2958e235c5b0944376f4c4983ceb026b4da962deb10b249f315
-size 19326
+oid sha256:195cb0e133075a7c8d2b429c32337dd62a7ce97ede7c0a62b630f03eaa145f63
+size 19293
diff --git a/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer_null_TextComposerVoice-D-4_4_null,NEXUS_5,1.0,en].png b/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer_null_TextComposerVoice-D-4_4_null,NEXUS_5,1.0,en].png
@@ -1,3 +1,3 @@
 version https://git-lfs.github.com/spec/v1
-oid sha256:5abbadadbc4b0943be47d269cddecfe42e8a7ae43e4dd618eb2a8d5aefc7c1b1
-size 27291
+oid sha256:d93f97e1ae36edc6375b00de4421166f8012ab16cc6629897b6a36b062cbbff6
+size 27255
diff --git a/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer_null_TextComposerVoice-N-4_5_null,NEXUS_5,1.0,en].png b/tests/uitests/src/test/snapshots/images/ui_S_t[l.textcomposer_null_TextComposerVoice-N-4_5_null,NEXUS_5,1.0,en].png
@@ -1,3 +1,3 @@
 version https://git-lfs.github.com/spec/v1
-oid sha256:f73f8eed6746a54bd0bf706acda638155ea99e7aba8b78fc809b9f26b0a5986f
-size 24901
+oid sha256:b34b9934ea8c06dc98b080d6dcd07c2c12d95c0841c1fd09e2ebc62bb8df2fce
+size 24904

Original file line number	Diff line number	Diff line change
`@@ -129,12 +129,14 @@ private fun PlayerButton(`
`129`	`129`	`private fun PauseIcon() = Icon(`
`130`	`130`	`resourceId = R.drawable.ic_pause,`
`131`	`131`	`contentDescription = stringResource(id = CommonStrings.a11y_pause),`
	`132`	`+ tint = ElementTheme.colors.iconSecondary,`
`132`	`133`	`)`
`133`	`134`
`134`	`135`	`@Composable`
`135`	`136`	`private fun PlayIcon() = Icon(`
`136`	`137`	`resourceId = R.drawable.ic_play,`
`137`	`138`	`contentDescription = stringResource(id = CommonStrings.a11y_play),`
	`139`	`+ tint = ElementTheme.colors.iconSecondary,`
`138`	`140`	`)`
`139`	`141`
`140`	`142`	`@PreviewsDayNight`