gsadmin/meet
1
0
Fork 0
mirror of https://github.com/suitenumerique/meet.git synced 2026-08-31 04:37:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

⚡️(frontend) narrow ParticipantToggle event subscriptions

Browse Source 
Reduce the set of events the ParticipantToggle subscribes to, so it
does not re-render on every LocalParticipant event but only on the
ones that actually affect its rendering.
This commit is contained in:
lebaudantoine
2026-07-14 19:13:32 +02:00
parent 0285870ccb
commit 430e638f62
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/frontend/src/features/rooms/livekit/components/controls/Participants/ParticipantsToggle.tsx
+9 -3
View File
@@ -3,10 +3,11 @@ import { RiGroupLine, RiInfinityLine } from '@remixicon/react'
import { ToggleButton } from '@/primitives' import { ToggleButton } from '@/primitives'
import { VisualOnlyTooltip } from '@/primitives/VisualOnlyTooltip' import { VisualOnlyTooltip } from '@/primitives/VisualOnlyTooltip'
import { css } from '@/styled-system/css' import { css } from '@/styled-system/css'
import { useParticipants } from '@livekit/components-react' import { useRemoteParticipants } from '@livekit/components-react'
import { useSidePanel } from '../../../hooks/useSidePanel' import { useSidePanel } from '../../../hooks/useSidePanel'
import { ToggleButtonProps } from '@/primitives/ToggleButton' import { ToggleButtonProps } from '@/primitives/ToggleButton'
import { useRegisterKeyboardShortcut } from '@/features/shortcuts/useRegisterKeyboardShortcut' import { useRegisterKeyboardShortcut } from '@/features/shortcuts/useRegisterKeyboardShortcut'
import { RoomEvent } from 'livekit-client'
export const ParticipantsToggle = ({ export const ParticipantsToggle = ({
onPress, onPress,
@@ -19,8 +20,13 @@ export const ParticipantsToggle = ({
* The 'numParticipant' property on the room only updates when the room's metadata changes, * The 'numParticipant' property on the room only updates when the room's metadata changes,
* resulting in a delay compared to the participant list's actual refresh rate. * resulting in a delay compared to the participant list's actual refresh rate.
*/ */
const participants = useParticipants() const remoteParticipants = useRemoteParticipants({
const numParticipants = participants?.length updateOnlyOn: [
RoomEvent.ParticipantConnected,
RoomEvent.ParticipantDisconnected,
],
})
const numParticipants = remoteParticipants?.length + 1 // for the local participant
const announcedCount = const announcedCount =
numParticipants && numParticipants > 0 ? numParticipants : 1 numParticipants && numParticipants > 0 ? numParticipants : 1