From 184e9928f5dbda9c9affe5aa5628b758309536bb Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Tue, 19 Dec 2023 18:26:41 +0100 Subject: [PATCH] Use CompoundIcon and remove ic_send.xml --- .../libraries/designsystem/icons/IconsList.kt | 1 - .../src/main/res/drawable/ic_send.xml | 25 ------------------- .../textcomposer/components/SendButton.kt | 15 +++++------ 3 files changed, 8 insertions(+), 33 deletions(-) delete mode 100644 libraries/designsystem/src/main/res/drawable/ic_send.xml diff --git a/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/icons/IconsList.kt b/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/icons/IconsList.kt index 8585ad9339..a89bfbfa4f 100644 --- a/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/icons/IconsList.kt +++ b/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/icons/IconsList.kt @@ -45,7 +45,6 @@ internal val iconsOther = listOf( R.drawable.ic_plus, R.drawable.ic_quote, R.drawable.ic_reply, - R.drawable.ic_send, R.drawable.ic_sign_out, R.drawable.ic_strikethrough, R.drawable.ic_take_photo_camera, diff --git a/libraries/designsystem/src/main/res/drawable/ic_send.xml b/libraries/designsystem/src/main/res/drawable/ic_send.xml deleted file mode 100644 index 8b90be336e..0000000000 --- a/libraries/designsystem/src/main/res/drawable/ic_send.xml +++ /dev/null @@ -1,25 +0,0 @@ - - - - - diff --git a/libraries/textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/components/SendButton.kt b/libraries/textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/components/SendButton.kt index 298feb0076..22c689deca 100644 --- a/libraries/textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/components/SendButton.kt +++ b/libraries/textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/components/SendButton.kt @@ -30,14 +30,15 @@ import androidx.compose.ui.draw.clip import androidx.compose.ui.graphics.Color import androidx.compose.ui.res.stringResource import androidx.compose.ui.unit.dp +import io.element.android.compound.icons.CompoundIcons +import io.element.android.compound.icons.compoundicons.Check +import io.element.android.compound.icons.compoundicons.Send +import io.element.android.compound.theme.ElementTheme import io.element.android.libraries.designsystem.preview.ElementPreview import io.element.android.libraries.designsystem.preview.PreviewsDayNight import io.element.android.libraries.designsystem.theme.components.Icon import io.element.android.libraries.designsystem.theme.components.IconButton -import io.element.android.libraries.designsystem.utils.CommonDrawables import io.element.android.libraries.textcomposer.model.MessageComposerMode -import io.element.android.compound.theme.ElementTheme -import io.element.android.libraries.designsystem.icons.CompoundDrawables import io.element.android.libraries.ui.strings.CommonStrings @Composable @@ -53,9 +54,9 @@ internal fun SendButton( onClick = onClick, enabled = canSendMessage, ) { - val iconId = when (composerMode) { - is MessageComposerMode.Edit -> CompoundDrawables.ic_check - else -> CommonDrawables.ic_send + val iconVector = when (composerMode) { + is MessageComposerMode.Edit -> CompoundIcons.Check + else -> CompoundIcons.Send } val iconSize = when (composerMode) { is MessageComposerMode.Edit -> 24.dp @@ -81,7 +82,7 @@ internal fun SendButton( .height(iconSize) .padding(start = iconStartPadding) .align(Alignment.Center), - resourceId = iconId, + imageVector = iconVector, contentDescription = contentDescription, // Exception here, we use Color.White instead of ElementTheme.colors.iconOnSolidPrimary tint = if (canSendMessage) Color.White else ElementTheme.colors.iconDisabled