feature/23-voice-room/poc #93

Merged
hodlbod merged 68 commits from feature/23-voice-room/poc into dev 2026-03-16 20:38:06 +00:00
2 changed files with 8 additions and 0 deletions
Showing only changes of commit 1b99ed6704 - Show all commits
+5
View File
@@ -5,6 +5,7 @@
import SecondaryNavItem from "@lib/components/SecondaryNavItem.svelte"
import ProfileCircle from "@app/components/ProfileCircle.svelte"
import RoomName from "@app/components/RoomName.svelte"
import {isMobileViewport} from "@lib/html"
import {pushToast} from "@app/util/toast"
import {
deriveVoiceParticipants,
@@ -26,6 +27,10 @@
let joinAbortController: AbortController | undefined
const handleClick = async () => {
if (isMobileViewport()) {
pushToast({theme: "error", message: "Voice rooms are not yet supported on mobile."})
return
}
if (isActive) {
await leaveVoiceRoom()
hodlbod marked this conversation as resolved Outdated
Outdated
Review

Instead of this, we should just not render voice rooms on mobile

Instead of this, we should just not render voice rooms on mobile
Outdated
Review

I tried that, but realized that you could then join a call on desktop web, resize your browser to be small and then lose the UI to leave the call. As I type it out I realize that's too edgy of a case 😂 I will just hide it.

I tried that, but realized that you could then join a call on desktop web, resize your browser to be small and then lose the UI to leave the call. As I type it out I realize that's too edgy of a case 😂 I will just hide it.
return
8
+3
View File
@@ -80,6 +80,9 @@ export const createScroller = ({
export const isMobile = "ontouchstart" in document.documentElement
// Remove this when we implement voice rooms on mobile
export const isMobileViewport = () => window.innerWidth <= 768
export const downloadText = (filename: string, text: string) => {
const blob = new Blob([text], {type: "text/plain"})
const url = URL.createObjectURL(blob)