From aeb6cddc2f150881ff59d9244d53604f5f3ceb5d Mon Sep 17 00:00:00 2001 From: Robin Date: Fri, 1 Apr 2022 11:28:47 -0400 Subject: [PATCH] Update video rooms to new design specs (#21623) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * "Voice room" → "video room" * Customize Jitsi behavior in video rooms --- docs/labs.md | 4 ++-- src/vector/jitsi/index.ts | 23 +++++++++++++++++++++-- 2 files changed, 23 insertions(+), 4 deletions(-) diff --git a/docs/labs.md b/docs/labs.md index e6e74db94c..72b8b09bae 100644 --- a/docs/labs.md +++ b/docs/labs.md @@ -183,6 +183,6 @@ Threads can be access by clicking their summary below the root event on the room This feature might work in degraded mode if the homeserver a user is connected to does not advertise support for the unstable feature `org.matrix.msc3440` when calling the `/versions` API endpoint. -## Voice & video rooms (`feature_voice_rooms`) [In Development] +## Voice & video rooms (`feature_video_rooms`) [In Development] -Enables support for creating and joining voice & video rooms, which are persistent voice chats that users can jump in and out of. +Enables support for creating and joining video rooms, which are persistent video chats that users can jump in and out of. diff --git a/src/vector/jitsi/index.ts b/src/vector/jitsi/index.ts index d62b76f5ac..598d4f3e05 100644 --- a/src/vector/jitsi/index.ts +++ b/src/vector/jitsi/index.ts @@ -51,6 +51,7 @@ let roomId: string; let openIdToken: IOpenIDCredentials; let roomName: string; let startAudioOnly: boolean; +let isVideoChannel: boolean; let widgetApi: WidgetApi; let meetApi: any; // JitsiMeetExternalAPI @@ -120,12 +121,13 @@ const ack = (ev: CustomEvent) => widgetApi.transport.reply(ev roomId = qsParam('roomId', true); roomName = qsParam('roomName', true); startAudioOnly = qsParam('isAudioOnly', true) === "true"; + isVideoChannel = qsParam('isVideoChannel', true) === "true"; // We've reached the point where we have to wait for the config, so do that then parse it. const instanceConfig = new SnakedObject((await configPromise) ?? {}); const jitsiConfig = instanceConfig.get("jitsi_widget") ?? {}; skipOurWelcomeScreen = (new SnakedObject(jitsiConfig)) - .get("skip_built_in_welcome_screen") || false; + .get("skip_built_in_welcome_screen") || isVideoChannel; // If we're meant to skip our screen, skip to the part where we show Jitsi instead of us. // We don't set up the call yet though as this might lead to failure without the widget API. @@ -300,6 +302,7 @@ function joinConference() { // event handler bound in HTML "they mention 'external_api' or 'jitsi' in the stack. They're just Jitsi Meet trying to parse " + "our fragment values and not recognizing the options.", ); + const options = { width: "100%", height: "100%", @@ -313,10 +316,23 @@ function joinConference() { // event handler bound in HTML }, configOverwrite: { startAudioOnly, - }, + } as any, jwt: jwt, }; + // Video channel widgets need some more tailored config options + if (isVideoChannel) { + // Ensure that we start on Jitsi Meet's native prejoin screen, for + // deployments that skip straight to the conference by default + options.configOverwrite.prejoinConfig = { enabled: true }; + // Use a simplified set of toolbar buttons + options.configOverwrite.toolbarButtons = [ + "microphone", "camera", "desktop", "tileview", "hangup", + ]; + // Hide all top bar elements + options.configOverwrite.conferenceInfo = { autoHide: [] }; + } + meetApi = new JitsiMeetExternalAPI(jitsiDomain, options); if (displayName) meetApi.executeCommand("displayName", displayName); if (avatarUrl) meetApi.executeCommand("avatarUrl", avatarUrl); @@ -332,6 +348,9 @@ function joinConference() { // event handler bound in HTML widgetApi.setAlwaysOnScreen(true); widgetApi.transport.send(ElementWidgetActions.JoinCall, {}); } + + // Video rooms should start in tile mode + if (isVideoChannel) meetApi.executeCommand("setTileView", true); }); meetApi.on("readyToClose", () => {