Bladeren bron

Apply suggestions from code review

Co-authored-by: Saúl Ibarra Corretgé <saghul@jitsi.org>
master
Hristo Terezov 4 jaren geleden
bovenliggende
commit
bb670fd90d
1 gewijzigde bestanden met toevoegingen van 3 en 3 verwijderingen
  1. 3
    3
      react/features/filmstrip/components/native/Thumbnail.js

+ 3
- 3
react/features/filmstrip/components/native/Thumbnail.js Bestand weergeven

330
         = getTrackByMediaTypeAndParticipant(tracks, MEDIA_TYPE.AUDIO, id);
330
         = getTrackByMediaTypeAndParticipant(tracks, MEDIA_TYPE.AUDIO, id);
331
     const videoTrack
331
     const videoTrack
332
         = getTrackByMediaTypeAndParticipant(tracks, MEDIA_TYPE.VIDEO, id);
332
         = getTrackByMediaTypeAndParticipant(tracks, MEDIA_TYPE.VIDEO, id);
333
-    const videoMuted = !videoTrack || videoTrack.muted;
334
-    const isScreenShare = videoTrack && videoTrack.videoType === VIDEO_TYPE.DESKTOP;
333
+    const videoMuted = videoTrack?.muted ?? true;
334
+    const isScreenShare = videoTrack?.videoType === VIDEO_TYPE.DESKTOP;
335
     const participantCount = getParticipantCount(state);
335
     const participantCount = getParticipantCount(state);
336
     const renderDominantSpeakerIndicator = participant && participant.dominantSpeaker && participantCount > 2;
336
     const renderDominantSpeakerIndicator = participant && participant.dominantSpeaker && participantCount > 2;
337
     const _isEveryoneModerator = isEveryoneModerator(state);
337
     const _isEveryoneModerator = isEveryoneModerator(state);
338
     const renderModeratorIndicator = !_isEveryoneModerator
338
     const renderModeratorIndicator = !_isEveryoneModerator
339
-        && participant && participant.role === PARTICIPANT_ROLE.MODERATOR;
339
+        && participant?.role === PARTICIPANT_ROLE.MODERATOR;
340
     const participantInLargeVideo = id === largeVideo.participantId;
340
     const participantInLargeVideo = id === largeVideo.participantId;
341
 
341
 
342
     return {
342
     return {

Laden…
Annuleren
Opslaan