From d5faa5ea90f2fc3a313633dbf39c44919e0ddffa Mon Sep 17 00:00:00 2001
From: Robin <robin@robin.town>
Date: Fri, 26 Jul 2024 06:51:34 -0400
Subject: [PATCH] Don't show the speaker in the spotlight in large grids
 (#2511)

We've concluded that this behavior is actually more distracting than it is helpful, and we want to try out what it's like to just have the importance ordering and visual cues help you find who's speaking.
---
 src/state/CallViewModel.ts | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/src/state/CallViewModel.ts b/src/state/CallViewModel.ts
index 3cb6ab271..7d5408582 100644
--- a/src/state/CallViewModel.ts
+++ b/src/state/CallViewModel.ts
@@ -75,13 +75,6 @@ import { duplicateTiles } from "../settings/settings";
 // list again
 const POST_FOCUS_PARTICIPANT_UPDATE_DELAY_MS = 3000;
 
-// This is the number of participants that we think constitutes a "large" grid.
-// The hypothesis is that, after this many participants there's enough cognitive
-// load that it makes sense to show the speaker in an easy-to-locate spotlight
-// tile. We might change this to a scroll-based condition or do something else
-// entirely with the spotlight tile, if we workshop this further.
-const largeGridThreshold = 20;
-
 export interface GridLayout {
   type: "grid";
   spotlight?: MediaViewModel[];
@@ -614,10 +607,7 @@ export class CallViewModel extends ViewModel {
                         : {
                             type: "grid",
                             spotlight:
-                              screenShares.length > 0 ||
-                              grid.length > largeGridThreshold
-                                ? spotlight
-                                : undefined,
+                              screenShares.length > 0 ? spotlight : undefined,
                             grid,
                           },
                   );