Skip to content

Commit

Permalink
Merge pull request #26879 from element-hq/toger5/jitsi-video-channel-…
Browse files Browse the repository at this point in the history
…lobby

Use jitsi-lobby in video channel (video rooms)
  • Loading branch information
toger5 authored Jan 23, 2024
2 parents db78839 + 8ab44c9 commit 41576e4
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/vector/jitsi/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -292,9 +292,7 @@ function switchVisibleContainers(): void {

function toggleConferenceVisibility(inConference: boolean): void {
document.getElementById("jitsiContainer")!.style.visibility = inConference ? "unset" : "hidden";
// Video rooms have a separate UI for joining, so they should never show our join button
document.getElementById("joinButtonContainer")!.style.visibility =
inConference || isVideoChannel ? "hidden" : "unset";
document.getElementById("joinButtonContainer")!.style.visibility = inConference ? "hidden" : "unset";
}

function skipToJitsiSplashScreen(): void {
Expand Down Expand Up @@ -450,9 +448,8 @@ async function joinConference(audioInput?: string | null, videoInput?: string |

// Video channel widgets need some more tailored config options
if (isVideoChannel) {
// Ensure that we skip Jitsi Meet's native prejoin screen, for
// deployments that have it enabled
options.configOverwrite!.prejoinConfig = { enabled: false };
// We don't skip jitsi's prejoin screen for video rooms.
options.configOverwrite!.prejoinConfig = { enabled: true };
// Use a simplified set of toolbar buttons
options.configOverwrite!.toolbarButtons = ["microphone", "camera", "tileview", "hangup"];
// Note: We can hide the screenshare button in video rooms but not in
Expand Down

0 comments on commit 41576e4

Please sign in to comment.