```@Composable fun list( item: GymContents, onEditClick: @Composable () -> Unit, onDe...

# android

[Flammer99]

@Composable
fun list(
    item: GymContents,
    onEditClick: @Composable () -> Unit,
    onDeleteClick: () -> Unit,
    viewModel: GymViewModel
) {
    val isEditing by viewModel.isEditing.collectAsState()
    Row(
        modifier = Modifier
            .padding(8.dp)
            .fillMaxWidth()
            .border(
                border = BorderStroke(2.dp, Color.Black),
                shape = RoundedCornerShape(20)
            )
    ) {
        Text(text = item.name, modifier = Modifier.padding(8.dp))
        Text(
            text = "Sets: ${item.sets}",
            modifier = Modifier.padding(8.dp)
        )
        Text(text = "Reps: ${item.reps}")
        Text(text = "Lift: ${item.lift}")
        Row(modifier = Modifier.padding(8.dp)) {
            IconButton(onClick = onEditClick) {
                Icon(imageVector = Icons.Default.Edit, contentDescription = null)
            }

            IconButton(onClick = onDeleteClick) {
                Icon(imageVector = Icons.Default.Delete, contentDescription = null)
            }

        }
    }}

Type inference failed. Expected type mismatch: inferred type is @Composable () -> Unit but () -> Unit was expected

[Chrimaeon]

You defined onEditClick as a Composable function. But onClick expects a “normal” function.

[Flammer99]

mh

[Flammer99]

so what do u think i should go for?

[Chrimaeon]

https://developer.android.com/develop/ui/compose/state#state-hoisting

[Chrimaeon]

https://developer.android.com/develop/ui/compose/state-hoisting#ui-logic

[Chrimaeon]

Also you might get better infos of you ask questions about compose in #compose-android

[Flammer99]

okk

[Julien Raad]

On edit click cannot be compostable. You need to manage showing/hiding the dialog via a state.
To do that onEditClick should just change that state from false to true.
Something like var isDialogVisible by rememberSaveable {
mutableStateOf(false) } then onEditClick = { isDialogVisible = true }

In your compose code, do
if (isDialogVisible) {
EditDialog() // that's the composable function
}

[Flammer99]

done it, but in this way i think that the edit function is not called

[Chrimaeon]

for reference to the cross-post https://kotlinlang.slack.com/archives/C04TPPEQKEJ/p1711986317997799