Support long-click on video preview

Author: grzesiek2010

collect_app/src/main/java/org/odk/collect/android/widgets/WidgetAnswerFactory.kt

@@ -12,11 +12,12 @@ fun WidgetAnswer(
     modifier: Modifier = Modifier,
     prompt: FormEntryPrompt,
     answer: String?,
-    viewModelProvider: ViewModelProvider
+    viewModelProvider: ViewModelProvider,
+    onLongClick: () -> Unit = {}
 ) {
     if (answer != null) {
         when (prompt.controlType) {
-            Constants.CONTROL_VIDEO_CAPTURE -> VideoWidgetAnswer.Container(modifier, answer, viewModelProvider)
+            Constants.CONTROL_VIDEO_CAPTURE -> VideoWidgetAnswer.Container(modifier, answer, viewModelProvider, onLongClick)
             else -> throw IllegalArgumentException("Unsupported control type: ${prompt.controlType}")
         }
     }

collect_app/src/main/java/org/odk/collect/android/widgets/video/ExVideoWidget.kt

@@ -68,13 +68,14 @@ class ExVideoWidget(
                     readOnly,
                     buttonFontSize,
                     onLaunchClick = { launchExternalApp() },
-                    onLongClick = { this.showContextMenu() }
+                    onLongClick = { showContextMenu() }
                 ) {
                     WidgetAnswer(
                         Modifier.padding(top = dimensionResource(id = dimen.margin_standard)),
                         formEntryPrompt,
                         binaryName,
-                        viewModelProvider
+                        viewModelProvider,
+                        onLongClick = { showContextMenu() }
                     )
                 }
             }

collect_app/src/main/java/org/odk/collect/android/widgets/video/VideoWidget.kt

@@ -83,13 +83,14 @@ class VideoWidget(
                         )
                     },
                     onChooseClick = { chooseVideo() },
-                    onLongClick = { this.showContextMenu() }
+                    onLongClick = { showContextMenu() }
                 ) {
                     WidgetAnswer(
                         Modifier.padding(top = dimensionResource(id = dimen.margin_standard)),
                         formEntryPrompt,
                         binaryName,
-                        viewModelProvider
+                        viewModelProvider,
+                        onLongClick = { showContextMenu() }
                     )
                 }
             }

collect_app/src/main/java/org/odk/collect/android/widgets/video/VideoWidgetAnswer.kt

@@ -2,7 +2,7 @@ package org.odk.collect.android.widgets.video
 
 import androidx.compose.foundation.Image
 import androidx.compose.foundation.background
-import androidx.compose.foundation.clickable
+import androidx.compose.foundation.combinedClickable
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
@@ -26,38 +26,49 @@ import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
 import androidx.lifecycle.ViewModelProvider
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
+import org.odk.collect.androidshared.ui.multiclicksafe.MultiClickGuard
 
 object VideoWidgetAnswer {
     @Composable
     fun Container(
         modifier: Modifier,
         answer: String,
-        viewModelProvider: ViewModelProvider
+        viewModelProvider: ViewModelProvider,
+        onLongClick: () -> Unit
     ) {
         val context = LocalContext.current
         val viewModel = viewModelProvider[VideoWidgetAnswerViewModel::class]
 
         val bitmapFlow = remember(answer) { viewModel.getFrame(answer, context) }
         val bitmap by bitmapFlow.collectAsStateWithLifecycle()
 
-        Content(modifier, bitmap) {
-            viewModel.playVideo(context, answer)
-        }
+        Content(
+            modifier,
+            bitmap,
+            onPlayClick = { viewModel.playVideo(context, answer) },
+            onLongClick = onLongClick
+        )
     }
 
     @Composable
     fun Content(
         modifier: Modifier,
         bitmap: ImageBitmap?,
-        onPlayClick: () -> Unit
+        onPlayClick: () -> Unit,
+        onLongClick: () -> Unit
     ) {
         Box(
             modifier = modifier
                 .fillMaxWidth()
                 .heightIn(max = 200.dp)
                 .clip(MaterialTheme.shapes.large)
-                .clickable(
-                    onClick = onPlayClick,
+                .combinedClickable(
+                    onClick = {
+                        if (MultiClickGuard.allowClick()) {
+                            onPlayClick()
+                        }
+                    },
+                    onLongClick = onLongClick,
                     onClickLabel = stringResource(org.odk.collect.strings.R.string.play_video)
                 ),
             contentAlignment = Alignment.Center