Remove default parameter values and make them not nullable.

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/customreaction/EmojiItem.kt

@@ -70,13 +70,13 @@ fun EmojiItem(
     item: Emoji,
     isSelected: Boolean,
     onSelectEmoji: (Emoji) -> Unit,
+    onLongPress: (Emoji) -> Unit,
+    onDismissSkinPicker: () -> Unit,
+    skinPickerEmoji: Emoji?,
+    selectedSkinUnicodes: ImmutableSet<String>,
+    hasSelectedSkin: Boolean,
     modifier: Modifier = Modifier,
     emojiSize: TextUnit = 20.sp,
-    onLongPress: ((Emoji) -> Unit)? = null,
-    skinPickerEmoji: Emoji? = null,
-    onDismissSkinPicker: (() -> Unit)? = null,
-    selectedSkinUnicodes: ImmutableSet<String> = persistentSetOf(),
-    hasSelectedSkin: Boolean = false,
 ) {
     val backgroundColor = when {
         isSelected -> ElementTheme.colors.bgActionPrimaryRest
@@ -139,12 +139,10 @@ fun EmojiItem(
                         var startOffset = Offset.Zero
                         detectDragGesturesAfterLongPress(
                             onDragStart = { position ->
-                                if (onLongPress != null) {
+                                startOffset = position
-                                    startOffset = position
+                                dismissed = false
-                                    dismissed = false
+                                onLongPress(item)
-                                    onLongPress(item)
+                                hoveredIndex = -1
-                                    hoveredIndex = -1
-                                }
                             },
                             onDrag = { change, _ ->
                                 if (!dismissed) {
@@ -159,7 +157,7 @@ fun EmojiItem(
                                     if (!isValidDrag) {
                                         dismissed = true
                                         hoveredIndex = -1
-                                        onDismissSkinPicker?.invoke()
+                                        onDismissSkinPicker()
                                     } else {
                                         val skinItemsCount = item.skins!!.size
                                         // Original + variants
@@ -199,7 +197,7 @@ fun EmojiItem(
                                 }
                                 dismissed = false
                                 hoveredIndex = -1
-                                onDismissSkinPicker?.invoke()
+                                onDismissSkinPicker()
                             },
                             onDragCancel = {
                                 dismissed = false
@@ -241,7 +239,7 @@ fun EmojiItem(
             val pickerHeight = SkinTonePadding * 2 + SkinToneSlotSize
             val popupOffsetPx = with(density) { -pickerHeight.roundToPx() }
             Popup(
-                onDismissRequest = { onDismissSkinPicker?.invoke() },
+                onDismissRequest = { onDismissSkinPicker() },
                 alignment = Alignment.BottomCenter,
                 offset = IntOffset(0, popupOffsetPx),
             ) {
@@ -249,7 +247,7 @@ fun EmojiItem(
                     emoji = item,
                     onSelect = { selectedEmoji ->
                         onSelectEmoji(selectedEmoji)
-                        onDismissSkinPicker?.invoke()
+                        onDismissSkinPicker()
                     },
                     hoveredIndex = hoveredIndex,
                     selectedUnicodes = selectedSkinUnicodes,
@@ -282,6 +280,10 @@ internal fun EmojiItemPreview() = ElementPreview {
                         isSelected = isSelected,
                         onSelectEmoji = {},
                         hasSelectedSkin = hasSelectedSkin,
+                        onLongPress = {},
+                        onDismissSkinPicker = {},
+                        skinPickerEmoji = null,
+                        selectedSkinUnicodes = persistentSetOf(),
                     )
                 }
             }

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/customreaction/picker/EmojiPicker.kt

@@ -131,10 +131,10 @@ private fun EmojiResults(
     emojis: ImmutableList<Emoji>,
     isEmojiSelected: (Emoji) -> Boolean,
     onSelectEmoji: (Emoji) -> Unit,
-    onLongPress: ((Emoji) -> Unit)? = null,
+    onLongPress: (Emoji) -> Unit,
-    skinPickerEmoji: Emoji? = null,
+    skinPickerEmoji: Emoji?,
-    onDismissSkinPicker: (() -> Unit)? = null,
+    onDismissSkinPicker: () -> Unit,
-    selectedEmojis: ImmutableSet<String> = persistentSetOf(),
+    selectedEmojis: ImmutableSet<String>,
 ) {
     LazyVerticalGrid(
         modifier = Modifier.fillMaxSize(),

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/customreaction/picker/SkinTonePicker.kt

@@ -43,10 +43,10 @@ val SkinTonePadding = 4.dp
 @Composable
 fun SkinTonePicker(
     emoji: Emoji,
+    hoveredIndex: Int,
+    selectedUnicodes: ImmutableSet<String>,
     onSelect: (Emoji) -> Unit,
     modifier: Modifier = Modifier,
-    hoveredIndex: Int = -1,
-    selectedUnicodes: ImmutableSet<String> = persistentSetOf(),
 ) {
     val skins = emoji.skins.orEmpty()
     val emojiSize = 32.dp.toSp()
@@ -122,6 +122,8 @@ internal fun SkinTonePickerPreview() = ElementPreview {
             unicode = "👍",
             skins = aSkinList(),
         ),
+        hoveredIndex = -1,
+        selectedUnicodes = persistentSetOf(),
         onSelect = {},
     )
 }