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.
This commit is contained in:
@@ -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,
|
||||
},
|
||||
);
|
||||
|
||||
Reference in New Issue
Block a user