coracle/flotilla
1
3
Fork 9
Code Issues 31 Pull Requests 1 Actions Packages Projects 5 Releases Wiki Activity

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
Conversation 28 Commits 68 Files Changed 28
+18 -5
1 changed files with 18 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e0d2988877 - Show all commits
src/app/components/VoiceWidget.svelte
+18 -5
View File
@@ -3,10 +3,14 @@
import Microphone from "@assets/icons/microphone.svg?dataurl"
import MicrophoneOff from "@assets/icons/microphone-off.svg?dataurl"
import PhoneRounded from "@assets/icons/phone-rounded.svg?dataurl"
import InfoCircle from "@assets/icons/info-circle.svg?dataurl"
import Icon from "@lib/components/Icon.svelte"
import Button from "@lib/components/Button.svelte"
import {displayRoom} from "@app/core/state"
import {get} from "svelte/store"
import {currentVoiceSession, leaveVoiceRoom, toggleMute} from "@app/voice"
import RoomDetail from "@app/components/RoomDetail.svelte"
import {pushModal} from "@app/util/modal"
const roomName = $derived(
$currentVoiceSession ? displayRoom($currentVoiceSession.url, $currentVoiceSession.h) : "",
@@ -15,15 +19,24 @@
const handleDisconnect = () => leaveVoiceRoom()
const handleToggleMute = () => toggleMute()
hodlbod marked this conversation as resolved Outdated
hodlbod commented 2026-03-03 23:27:06 +00:00
Outdated
Review
Copy Link
Copy Source

Don't do indirection like this, just call the actual function

Don't do indirection like this, just call the actual function
👍 1
hodlbod commented 2026-03-05 22:47:21 +00:00
Outdated
Review
Copy Link
Copy Source

A stupid way to do this that I like would be ifLet($currentVoiceSession, ({url, h}) => pushModal(RoomDetail, {url, h})) but you don't have to do that

A stupid way to do this that I like would be `ifLet($currentVoiceSession, ({url, h}) => pushModal(RoomDetail, {url, h}))` but you don't have to do that
mplorentz commented 2026-03-09 14:45:35 +00:00
Outdated
Review
Copy Link
Copy Source

It is harder to read at first glance but ifLet brings warm feelings from better programming languages so it's done.

It is harder to read at first glance but `ifLet` brings warm feelings from better programming languages so it's done.
const showRoomDetail = () => {
const session = get(currentVoiceSession)
if (session) pushModal(RoomDetail, {url: session.url, h: session.h})
hodlbod marked this conversation as resolved Outdated
hodlbod commented 2026-03-03 23:28:31 +00:00
Outdated
Review
Copy Link
Copy Source

$currentVoiceSession is better than get

$currentVoiceSession is better than `get`
}
</script>
{#if $currentVoiceSession}
<div class="flex flex-col gap-2 rounded-box bg-base-100 p-3">
<div class="flex flex-col gap-0.5">
<span class="text-sm font-semibold text-success">Voice Connected</span>
<span class="ellipsize text-xs opacity-70">
{roomName} / {spaceName}
</span>
<div class="flex items-start justify-between gap-2">
<div class="min-w-0 flex flex-col gap-0.5">
<span class="text-sm font-semibold text-success">Voice Connected</span>
<span class="ellipsize text-xs opacity-70">
{roomName} / {spaceName}
</span>
</div>
<Button class="btn btn-sm btn-square btn-ghost shrink-0" onclick={showRoomDetail}>
<Icon icon={InfoCircle} size={4} />
</Button>
</div>
<div class="flex items-center gap-1">
<Button
3