Allow track placeholders to be pinned (#666)

livekit · Nov 1, 2023 · 01b0418 · 01b0418
1 parent 4f6159e
commit 01b0418
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/.changeset/three-parrots-swim.md b/.changeset/three-parrots-swim.md
@@ -0,0 +1,5 @@
+---
+"@livekit/components-react": patch
+---
+
+Allow track placeholders to be pinned
diff --git a/packages/react/src/context/pin-context.ts b/packages/react/src/context/pin-context.ts
@@ -1,11 +1,11 @@
-import type { PinState, TrackReference } from '@livekit/components-core';
+import type { PinState, TrackReferenceOrPlaceholder } from '@livekit/components-core';
 import type * as React from 'react';
 
 /** @internal */
 export type PinAction =
   | {
       msg: 'set_pin';
-      trackReference: TrackReference;
+      trackReference: TrackReferenceOrPlaceholder;
     }
   | { msg: 'clear_pin' };
 

diff --git a/packages/react/src/hooks/useFocusToggle.ts b/packages/react/src/hooks/useFocusToggle.ts
@@ -1,9 +1,5 @@
 import type { TrackReferenceOrPlaceholder } from '@livekit/components-core';
-import {
-  setupFocusToggle,
-  isTrackReferencePinned,
-  isTrackReference,
-} from '@livekit/components-core';
+import { setupFocusToggle, isTrackReferencePinned } from '@livekit/components-core';
 import type { Track, Participant } from 'livekit-client';
 import { useEnsureParticipant, useMaybeLayoutContext } from '../context';
 import { mergeProps } from '../mergeProps';
@@ -65,7 +61,7 @@ export function useFocusToggle({ trackRef, trackSource, participant, props }: Us
           props.onClick?.(event);
 
           // Set or clear focus based on current focus state.
-          if (trackRef && isTrackReference(trackRef)) {
+          if (trackRef) {
             if (inFocus) {
               layoutContext?.pin.dispatch?.({
                 msg: 'clear_pin',