Skip to content

Commit

Permalink
Don't show the speaker in the spotlight in large grids (#2511)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
robintown committed Jul 26, 2024
1 parent 5becd2e commit d5faa5e
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions src/state/CallViewModel.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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[];
Expand Down Expand Up @@ -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,
},
);
Expand Down

0 comments on commit d5faa5e

Please sign in to comment.