coracle/flotilla
1
3
Fork 9
Code Issues 32 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
+8 -2
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 63d9adef19 - Show all commits
src/app/components/VoiceWidget.svelte
+8 -2
View File
4
 @@ -40,11 +40,17 @@
</div>
<div class="flex items-center gap-1">
<Button
class="btn btn-sm btn-square {$currentVoiceSession.muted ? 'btn-error' : 'btn-ghost'}"
data-tip={$currentVoiceSession.muted ? "Unmute" : "Mute"}
class="center tooltip tooltip-top btn btn-sm btn-square {$currentVoiceSession.muted
? 'btn-error'
: 'btn-ghost'}"
hodlbod marked this conversation as resolved Outdated
hodlbod commented 2026-03-05 22:47:56 +00:00
Outdated
Review
Copy Link
Copy Source

If we have room here, it might be good to label the buttons with mute/leave

If we have room here, it might be good to label the buttons with mute/leave
mplorentz commented 2026-03-09 15:24:38 +00:00
Outdated
Review
Copy Link
Copy Source

There is not a ton of room. How about a hover state?

There is not a ton of room. How about a hover state?
hodlbod commented 2026-03-09 16:25:35 +00:00
Outdated
Review
Copy Link
Copy Source

Yeah, that's a fine compromise.

Yeah, that's a fine compromise.
onclick={toggleMute}>
<Icon icon={$currentVoiceSession.muted ? MicrophoneOff : Microphone} size={4} />
</Button>
<Button class="btn btn-sm btn-square btn-error" onclick={leaveVoiceRoom}>
<Button
data-tip="Leave room"
class="center tooltip tooltip-top btn btn-sm btn-square btn-error"
onclick={leaveVoiceRoom}>
<Icon icon={PhoneRounded} size={4} />
</Button>
</div>