- Disables saving m3u8 files (streaming can't be saved)

- Changes Error messages to not include stack traces and move them to a copy to clipboard action instead.

amethyst/src/main/java/com/vitorpamplona/amethyst/ui/actions/InformationDialog.kt

@@ -20,9 +20,12 @@
  */
 package com.vitorpamplona.amethyst.ui.actions
 
+import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.Row
 import androidx.compose.foundation.layout.Spacer
+import androidx.compose.foundation.layout.fillMaxWidth
+import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.text.selection.SelectionContainer
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.outlined.Done
@@ -32,9 +35,14 @@ import androidx.compose.material3.ButtonColors
 import androidx.compose.material3.ButtonDefaults
 import androidx.compose.material3.Icon
 import androidx.compose.material3.Text
+import androidx.compose.material3.TextButton
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.remember
 import androidx.compose.ui.Alignment
+import androidx.compose.ui.Modifier
+import androidx.compose.ui.platform.LocalClipboardManager
+import androidx.compose.ui.text.AnnotatedString
+import androidx.compose.ui.unit.dp
 import com.vitorpamplona.amethyst.R
 import com.vitorpamplona.amethyst.ui.stringRes
 import com.vitorpamplona.amethyst.ui.theme.Size16dp
@@ -50,15 +58,11 @@ fun InformationDialog(
     buttonColors: ButtonColors = ButtonDefaults.buttonColors(),
     onDismiss: () -> Unit,
 ) {
-    val stack = stringRes(id = R.string.stack)
+    val str = textContent ?: throwable.localizedMessage ?: throwable.message ?: throwable.javaClass.simpleName
-    val str =
+
+    val stack =
         remember(throwable) {
             val writer = StringWriter()
-            textContent?.let {
-                writer.append(it)
-                writer.append("\n\n")
-            }
-            writer.append(stack)
             writer.append("\n")
 
             throwable.printStackTrace(PrintWriter(writer))
@@ -66,13 +70,14 @@ fun InformationDialog(
             writer.toString()
         }
 
-    InformationDialog(title = title, textContent = str, buttonColors, onDismiss)
+    InformationDialog(title = title, textContent = str, moreInfo = stack, buttonColors, onDismiss)
 }
 
 @Composable
 fun InformationDialog(
     title: String,
     textContent: String,
+    moreInfo: String? = null,
     buttonColors: ButtonColors = ButtonDefaults.buttonColors(),
     onDismiss: () -> Unit,
 ) {
@@ -85,6 +90,22 @@ fun InformationDialog(
             }
         },
         confirmButton = {
+            Row(
+                modifier =
+                    Modifier
+                        .padding(all = 8.dp)
+                        .fillMaxWidth(),
+                horizontalArrangement = Arrangement.SpaceBetween,
+            ) {
+                moreInfo?.let {
+                    val clipboardManager = LocalClipboardManager.current
+                    TextButton(onClick = {
+                        clipboardManager.setText(AnnotatedString(it))
+                    }) {
+                        Text(stringRes(R.string.copy_stack_to_clipboard))
+                    }
+                }
+
                 Button(
                     onClick = onDismiss,
                     colors = buttonColors,
@@ -101,6 +122,7 @@ fun InformationDialog(
                         Text(stringRes(R.string.error_dialog_button_ok))
                     }
                 }
+            }
         },
     )
 }

amethyst/src/main/java/com/vitorpamplona/amethyst/ui/components/VideoView.kt

@@ -826,6 +826,7 @@ private fun RenderVideoPlayer(
                 keepPlaying.value = newKeepPlaying
             }
 
+            if (!videoUri.endsWith(".m3u8")) {
                 AnimatedSaveButton(controllerVisible, Modifier.align(Alignment.TopEnd).padding(end = Size110dp)) { context ->
                     saveImage(videoUri, mimeType, context, accountViewModel)
                 }
@@ -833,6 +834,11 @@ private fun RenderVideoPlayer(
                 AnimatedShareButton(controllerVisible, Modifier.align(Alignment.TopEnd).padding(end = Size165dp)) { popupExpanded, toggle ->
                     ShareImageAction(accountViewModel = accountViewModel, popupExpanded, videoUri, nostrUriCallback, null, null, null, mimeType, toggle)
                 }
+            } else {
+                AnimatedShareButton(controllerVisible, Modifier.align(Alignment.TopEnd).padding(end = Size110dp)) { popupExpanded, toggle ->
+                    ShareImageAction(accountViewModel = accountViewModel, popupExpanded, videoUri, nostrUriCallback, null, null, null, mimeType, toggle)
+                }
+            }
         } else {
             controller.volume = 0f
         }

amethyst/src/main/java/com/vitorpamplona/amethyst/ui/components/ZoomableContentDialog.kt

@@ -255,6 +255,7 @@ private fun DialogContent(
                     ShareImageAction(accountViewModel = accountViewModel, popupExpanded = popupExpanded, myContent, onDismiss = { popupExpanded.value = false })
                 }
 
+                if (myContent !is MediaUrlContent || !myContent.url.endsWith(".m3u8")) {
                     val localContext = LocalContext.current
 
                     val writeStoragePermissionState =
@@ -287,6 +288,7 @@ private fun DialogContent(
                 }
             }
         }
+    }
 }
 
 private fun saveImage(

amethyst/src/main/res/values/strings.xml

@@ -602,6 +602,8 @@
     <string name="automatically_show_url_preview_description">Show URL previews</string>
     <string name="load_image_description">When to load images</string>
 
+    <string name="copy_stack_to_clipboard">Copy Stack</string>
+
     <string name="copy_to_clipboard">Copy to clipboard</string>
     <string name="copy_npub_to_clipboard">Copy npub to clipboard</string>
     <string name="share_or_save">Share or Save</string>