From 3d7ee93fb30041ec571b60db8938659c624b0bad Mon Sep 17 00:00:00 2001
From: harshithpabbati <pabbatiharshith@gmail.com>
Date: Fri, 7 Jan 2022 14:17:08 +0530
Subject: [PATCH] Fix issue with screen share

---
 .../fitness-demo/components/Call/VideoView.js |  11 +-
 .../components/GridView/GridView.js           |  36 +++---
 .../ScreensAndPins/ScreenPinTile.js           |  42 +++++++
 .../ScreensAndPins/ScreensAndPins.js          | 105 ++++++++++++++++++
 .../SpeakerView/ScreensAndPins/index.js       |   1 +
 .../SpeakerView/SpeakerTile/SpeakerTile.js    |   5 +-
 .../components/SpeakerView/SpeakerView.js     |  14 ++-
 custom/fitness-demo/components/Tray/View.js   |   6 +-
 8 files changed, 186 insertions(+), 34 deletions(-)
 create mode 100644 custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreenPinTile.js
 create mode 100644 custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreensAndPins.js
 create mode 100644 custom/fitness-demo/components/SpeakerView/ScreensAndPins/index.js

diff --git a/custom/fitness-demo/components/Call/VideoView.js b/custom/fitness-demo/components/Call/VideoView.js
index 93e3118..167e6c5 100644
--- a/custom/fitness-demo/components/Call/VideoView.js
+++ b/custom/fitness-demo/components/Call/VideoView.js
@@ -1,12 +1,17 @@
-import React from 'react';
+import React, { useEffect } from 'react';
 import { useParticipants } from '@custom/shared/contexts/ParticipantsProvider';
 import { useUIState, VIEW_MODE_SPEAKER } from '@custom/shared/contexts/UIStateProvider';
 import { GridView } from '../GridView/GridView';
 import { SpeakerView } from '../SpeakerView';
 
 export const VideoView = () => {
-  const { viewMode } = useUIState();
-  const { participants } = useParticipants();
+  const { viewMode, setIsShowingScreenshare } = useUIState();
+  const { participants, screens } = useParticipants();
+
+  useEffect(() => {
+    const hasScreens = screens.length > 0;
+    setIsShowingScreenshare(hasScreens);
+  }, [screens, setIsShowingScreenshare]);
 
   if (!participants.length) return null;
   return viewMode === VIEW_MODE_SPEAKER ? <SpeakerView />: <GridView />;
diff --git a/custom/fitness-demo/components/GridView/GridView.js b/custom/fitness-demo/components/GridView/GridView.js
index 4a3fe25..dbd3440 100644
--- a/custom/fitness-demo/components/GridView/GridView.js
+++ b/custom/fitness-demo/components/GridView/GridView.js
@@ -295,25 +295,25 @@ export const GridView = ({
 
   return (
     <div ref={gridRef} className="grid">
-      <Button
-        className="page-button prev"
-        disabled={!(pages > 1 && page > 1)}
-        type="button"
-        onClick={handlePrevClick}
-      >
-        <IconArrow />
-      </Button>
-
+      {(pages > 1 && page > 1) && (
+        <Button
+          className="page-button prev"
+          type="button"
+          onClick={handlePrevClick}
+        >
+          <IconArrow />
+        </Button>
+      )}
       <div className="tiles">{tiles}</div>
-
-      <Button
-        className="page-button next"
-        disabled={!(pages > 1 && page < pages)}
-        type="button"
-        onClick={handleNextClick}
-      >
-        <IconArrow />
-      </Button>
+      {(pages > 1 && page < pages) && (
+        <Button
+          className="page-button next"
+          type="button"
+          onClick={handleNextClick}
+        >
+          <IconArrow />
+        </Button>
+      )}
 
       <style jsx>{`
         .grid {
diff --git a/custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreenPinTile.js b/custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreenPinTile.js
new file mode 100644
index 0000000..8ffdbfe
--- /dev/null
+++ b/custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreenPinTile.js
@@ -0,0 +1,42 @@
+import { useState } from 'react';
+import Tile from '@custom/shared/components/Tile';
+
+export const ScreenPinTile = ({
+  height,
+  hideName = false,
+  item,
+  maxWidth,
+  ratio: initialRatio,
+}) => {
+  const [ratio, setRatio] = useState(initialRatio);
+  const handleResize = (aspectRatio) => setRatio(aspectRatio);
+
+  if (item.isScreenshare) {
+    return (
+      <Tile
+        aspectRatio={initialRatio}
+        hideName={hideName}
+        participant={item}
+        mirrored={false}
+        style={{
+          height,
+          maxWidth,
+        }}
+      />
+    );
+  }
+
+  return (
+    <Tile
+      aspectRatio={ratio}
+      participant={item}
+      onVideoResize={handleResize}
+      style={{
+        maxHeight: height,
+        maxWidth: height * ratio,
+      }}
+    />
+  );
+};
+
+export default ScreenPinTile;
\ No newline at end of file
diff --git a/custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreensAndPins.js b/custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreensAndPins.js
new file mode 100644
index 0000000..551ad9e
--- /dev/null
+++ b/custom/fitness-demo/components/SpeakerView/ScreensAndPins/ScreensAndPins.js
@@ -0,0 +1,105 @@
+import { useMemo, useRef, useState } from 'react';
+
+import { useCallState } from '@custom/shared/contexts/CallProvider';
+import { useUIState } from '@custom/shared/contexts/UIStateProvider';
+import { useResize } from '@custom/shared/hooks/useResize';
+import { useDeepCompareMemo } from 'use-deep-compare';
+import { ScreenPinTile } from './ScreenPinTile';
+
+const MAX_SCREENS_AND_PINS = 3;
+
+export const ScreensAndPins = ({ items }) => {
+  const { showNames } = useCallState();
+  const { pinnedId, sidebarView } = useUIState();
+  const viewRef = useRef(null);
+  const [dimensions, setDimensions] = useState({
+    width: 1,
+    height: 1,
+  });
+
+  useResize(() => {
+    const { width, height } = viewRef.current?.getBoundingClientRect();
+    setDimensions({
+      width,
+      height,
+    });
+  }, [viewRef, sidebarView]);
+
+  const visibleItems = useDeepCompareMemo(() => {
+    const isPinnedScreenshare = ({ id, isScreenshare }) =>
+      isScreenshare && id === pinnedId;
+    if (items.some(isPinnedScreenshare)) {
+      return items.filter(isPinnedScreenshare);
+    }
+    return items;
+  }, [items, pinnedId]);
+
+  const { height, maxWidth, aspectRatio } = useMemo(() => {
+    /**
+     * We're relying on calculating what there is room for
+     * for the total number of s+p tiles instead of using
+     * videoTrack.getSettings because (currently) getSettings
+     * is unreliable in Firefox.
+     */
+    const containerAR = dimensions.width / dimensions.height;
+    const maxItems = Math.min(visibleItems.length, MAX_SCREENS_AND_PINS);
+    const cols = Math.min(maxItems, Math.ceil(containerAR));
+    const rows = Math.ceil(visibleItems.length / cols);
+    const height = dimensions.height / rows;
+    const maxWidth = dimensions.width / cols;
+    return {
+      height,
+      maxWidth,
+      aspectRatio: maxWidth / height,
+    };
+  }, [dimensions, visibleItems?.length]);
+
+  return (
+    <div ref={viewRef}>
+      {visibleItems.map((item) => (
+        <div
+          className="tileWrapper"
+          key={item.id}
+          style={{
+            height,
+            maxWidth,
+          }}
+        >
+          <ScreenPinTile
+            height={height}
+            hideName={!showNames}
+            item={item}
+            maxWidth={maxWidth}
+            ratio={aspectRatio}
+          />
+        </div>
+      ))}
+      <style jsx>{`
+        div {
+          align-items: center;
+          display: flex;
+          flex-wrap: wrap;
+          height: 100%;
+          justify-content: center;
+          width: 100%;
+        }
+        div :global(.tileWrapper) {
+          background: var(--background);
+          align-items: center;
+          display: flex;
+          flex: none;
+          justify-content: center;
+          position: relative;
+          width: 100%;
+        }
+        div :global(.tile .content) {
+          margin: auto;
+          max-height: 100%;
+          max-width: 100%;
+        }
+      `}</style>
+    </div>
+  );
+};
+
+export default ScreensAndPins;
\ No newline at end of file
diff --git a/custom/fitness-demo/components/SpeakerView/ScreensAndPins/index.js b/custom/fitness-demo/components/SpeakerView/ScreensAndPins/index.js
new file mode 100644
index 0000000..228aada
--- /dev/null
+++ b/custom/fitness-demo/components/SpeakerView/ScreensAndPins/index.js
@@ -0,0 +1 @@
+export { ScreensAndPins } from './ScreensAndPins';
\ No newline at end of file
diff --git a/custom/fitness-demo/components/SpeakerView/SpeakerTile/SpeakerTile.js b/custom/fitness-demo/components/SpeakerView/SpeakerTile/SpeakerTile.js
index b651402..c69a8e8 100644
--- a/custom/fitness-demo/components/SpeakerView/SpeakerTile/SpeakerTile.js
+++ b/custom/fitness-demo/components/SpeakerView/SpeakerTile/SpeakerTile.js
@@ -18,10 +18,7 @@ export const SpeakerTile = ({ participant, screenRef }) => {
     setScreenHeight(rect.height);
   }, [screenRef]);
 
-  useResize(() => {
-    updateRatio();
-  }, [updateRatio]);
-
+  useResize(() => updateRatio(), [updateRatio]);
   useEffect(() => updateRatio(), [updateRatio]);
 
   /**
diff --git a/custom/fitness-demo/components/SpeakerView/SpeakerView.js b/custom/fitness-demo/components/SpeakerView/SpeakerView.js
index 2aaeb1c..3eccd37 100644
--- a/custom/fitness-demo/components/SpeakerView/SpeakerView.js
+++ b/custom/fitness-demo/components/SpeakerView/SpeakerView.js
@@ -1,13 +1,11 @@
 import React, { useEffect, useMemo, useRef } from 'react';
-import { Container } from '@custom/basic-call/components/Call/Container';
-import Header from '@custom/basic-call/components/Call/Header';
 import ParticipantBar from '@custom/shared/components/ParticipantBar/ParticipantBar';
-import VideoContainer from '@custom/shared/components/VideoContainer/VideoContainer';
 import { useCallState } from '@custom/shared/contexts/CallProvider';
 import { useParticipants } from '@custom/shared/contexts/ParticipantsProvider';
 import { useTracks } from '@custom/shared/contexts/TracksProvider';
 import { useUIState } from '@custom/shared/contexts/UIStateProvider';
 import { isScreenId } from '@custom/shared/contexts/participantsState';
+import { ScreensAndPins } from './ScreensAndPins';
 import { SpeakerTile } from './SpeakerTile';
 
 const SIDEBAR_WIDTH = 186;
@@ -40,10 +38,10 @@ export const SpeakerView = () => {
     return participants.length > 1 || hasScreenshares;
   }, [participants, pinnedId, screens]);
 
-  /* const screenShareTiles = useMemo(
+  const screenShareTiles = useMemo(
     () => <ScreensAndPins items={screensAndPinned} />,
     [screensAndPinned]
-  ); */
+  );
 
   const hasScreenshares = useMemo(() => screens.length > 0, [screens]);
 
@@ -80,7 +78,11 @@ export const SpeakerView = () => {
   return (
     <div className="speaker-view">
       <div ref={activeRef} className="active">
-        <SpeakerTile participant={currentSpeaker} screenRef={activeRef} />
+        {screensAndPinned.length > 0 ? (
+          screenShareTiles
+        ) : (
+          <SpeakerTile screenRef={activeRef} participant={currentSpeaker} />
+        )}
       </div>
       {showSidebar && (
         <ParticipantBar
diff --git a/custom/fitness-demo/components/Tray/View.js b/custom/fitness-demo/components/Tray/View.js
index 6740dbb..62828e0 100644
--- a/custom/fitness-demo/components/Tray/View.js
+++ b/custom/fitness-demo/components/Tray/View.js
@@ -10,14 +10,14 @@ export const ViewTray = () => {
   const { participants } = useParticipants();
   const { viewMode, setPreferredViewMode } = useUIState();
 
-  const onClick = () =>
-    setPreferredViewMode(viewMode === VIEW_MODE_SPEAKER ? VIEW_MODE_GRID: VIEW_MODE_SPEAKER);
+  const onViewClick = () =>
+    setPreferredViewMode(viewMode === VIEW_MODE_SPEAKER ? VIEW_MODE_GRID : VIEW_MODE_SPEAKER);
 
   return (
     <TrayButton
       label={viewMode === VIEW_MODE_GRID ? 'Speaker': 'Grid'}
       disabled={participants.length < 2}
-      onClick={onClick}
+      onClick={onViewClick}
     >
       {viewMode === VIEW_MODE_SPEAKER ? <IconGridView />: <IconSpeakerView />}
     </TrayButton>