From 3dcc05bf268b8d5323d6e79aa4b37f411cc009d3 Mon Sep 17 00:00:00 2001
From: John Livingston <git@john-livingston.fr>
Date: Wed, 3 Apr 2024 15:54:58 +0200
Subject: [PATCH] Share dialog: new page when no option is selected.

---
 client/videowatch/share.ts | 36 ++++++++++++++++++++++--------------
 1 file changed, 22 insertions(+), 14 deletions(-)

diff --git a/client/videowatch/share.ts b/client/videowatch/share.ts
index 526fcac7..357affe2 100644
--- a/client/videowatch/share.ts
+++ b/client/videowatch/share.ts
@@ -290,22 +290,30 @@ async function shareChatUrl (registerOptions: RegisterClientOptions, settings: a
     let shareStringValue
     let shareStringOpen: string | undefined
     if (!form.radioProtocolXMPP?.checked) {
-      shareStringValue = getIframeUri(registerOptions, settings, video, uriOptions)
-      if (form.generateIframe.checked) {
-        form.openButton.disabled = true
-        if (shareStringValue) {
-          // To properly escape all attributes, we are constructing an HTMLIframeElement
-          const iframe = document.createElement('iframe')
-          iframe.setAttribute('src', shareStringValue)
-          iframe.setAttribute('title', labelChatFor + ' ' + video.name)
-          iframe.setAttribute('sandbox', 'allow-same-origin allow-scripts allow-popups allow-forms')
-          iframe.setAttribute('width', '560')
-          iframe.setAttribute('height', '315')
-          iframe.setAttribute('frameborder', '0')
-          shareStringValue = iframe.outerHTML
+      if (form.readonly?.checked || form.autoColors?.checked || form.generateIframe?.checked) {
+        shareStringValue = getIframeUri(registerOptions, settings, video, uriOptions)
+        if (form.generateIframe.checked) {
+          form.openButton.disabled = true
+          if (shareStringValue) {
+            // To properly escape all attributes, we are constructing an HTMLIframeElement
+            const iframe = document.createElement('iframe')
+            iframe.setAttribute('src', shareStringValue)
+            iframe.setAttribute('title', labelChatFor + ' ' + video.name)
+            iframe.setAttribute('sandbox', 'allow-same-origin allow-scripts allow-popups allow-forms')
+            iframe.setAttribute('width', '560')
+            iframe.setAttribute('height', '315')
+            iframe.setAttribute('frameborder', '0')
+            shareStringValue = iframe.outerHTML
+          }
+        } else {
+          form.openButton.disabled = false
         }
       } else {
-        form.openButton.disabled = false
+        shareStringValue = window.location.protocol +
+          '//' +
+          window.location.host +
+          '/p/livechat/room?room=' +
+          encodeURIComponent(video.uuid)
       }
     } else {
       // we must generate a XMPP room address