Skip to content

Conversation

@konsalex
Copy link
Contributor

@konsalex konsalex commented Jan 29, 2026

Closes #244

Currently different impl for call-center vs if you are in a room.

Room's is cleaner, but there was no proper real estate in call-center now. We should rethink the "call-center" design around including the participant in 1-1 calls by the way.

CleanShot 2026-01-29 at 21 49 27@2x CleanShot 2026-01-29 at 21 49 49@2x

Summary by CodeRabbit

  • New Features
    • Added visual mute indicators for remote participants during calls, displaying a mic-off icon when a participant's microphone is disabled.
    • Integrated mute status indicators across call and room interfaces for improved communication awareness.

✏️ Tip: You can customize this high-level summary in your review settings.

@netlify
Copy link

netlify bot commented Jan 29, 2026

Deploy Preview for hoppdocs ready!

Name Link
🔨 Latest commit caa1dd7
🔍 Latest deploy log https://app.netlify.com/projects/hoppdocs/deploys/697bc8d3c5b9cd00078a36a3
😎 Deploy Preview https://deploy-preview-248--hoppdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Jan 29, 2026

📝 Walkthrough

Walkthrough

This pull request adds visual mute indicators for remote participants across the UI. The HoppAvatar component now displays a gray overlay with a microphone-off icon when the isMuted prop is true. Call center and Rooms components detect remote participant microphone state from LiveKit data and pass this information to avatar displays.

Changes

Cohort / File(s) Summary
Avatar Component Enhancement
tauri/src/components/ui/hopp-avatar.tsx
Added isMuted?: boolean prop to HoppAvatarProps interface. Conditionally renders absolute gray overlay with LuMicOff icon when isMuted is true. Imported LuMicOff from react-icons/lu.
Call Center Integration
tauri/src/components/ui/call-center.tsx
ConnectedActions now retrieves remote participants via useRemoteParticipants hook, matches remote audio participant by identity, derives isRemoteMuted from participant microphone state, and passes isMuted prop to HoppAvatar component.
Rooms Display
tauri/src/windows/main-window/tabs/Rooms.tsx
SelectedRoom now includes isMicrophoneEnabled flag from LiveKit participant data on participant entries. Renders Muted indicator with LuMicOff icon when participant microphone is disabled.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A whisker-twitch of muted friends,
Gray overlays where silence blends,
With mic icons dancing in the light,
Each avatar shows mute status right!
The remote voices hush their song,
We see who's quiet all along. 🔇

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'feat: new muted labels' is directly related to the main change—it accurately summarizes the introduction of visual mute indicators across three components (HoppAvatar, call-center, and Rooms), making the muted state of participants visible to users.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@iparaskev
Copy link
Contributor

LGTM!

@konsalex konsalex merged commit 63cbdbc into main Jan 30, 2026
16 checks passed
@iparaskev iparaskev deleted the muted-label branch January 31, 2026 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Add mute indication

3 participants