Add a way to maximize/pin widget from the PiP view

res/css/views/voip/_CallViewHeader.scss

@@ -43,6 +43,8 @@ limitations under the License.
 
 .mx_CallViewHeader_controls {
     margin-left: auto;
+    display: flex;
+    gap: 5px;
 }
 
 .mx_CallViewHeader_button {
@@ -61,17 +63,23 @@ limitations under the License.
         mask-size: contain;
         mask-position: center;
     }
-}
 
-.mx_CallViewHeader_button_fullscreen {
-    &::before {
-        mask-image: url('$(res)/img/element-icons/call/fullscreen.svg');
+    &.mx_CallViewHeader_button_fullscreen {
+        &::before {
+            mask-image: url('$(res)/img/element-icons/call/fullscreen.svg');
+        }
     }
-}
 
-.mx_CallViewHeader_button_expand {
-    &::before {
-        mask-image: url('$(res)/img/element-icons/call/expand.svg');
+    &.mx_CallViewHeader_button_pin {
+        &::before {
+            mask-image: url('$(res)/img/element-icons/room/pin-upright.svg');
+        }
+    }
+
+    &.mx_CallViewHeader_button_expand {
+        &::before {
+            mask-image: url('$(res)/img/element-icons/call/expand.svg');
+        }
     }
 }
 

src/components/views/voip/CallView.tsx

@@ -255,6 +255,13 @@ export default class CallView extends React.Component<IProps, IState> {
         return { primary, secondary };
     }
 
+    private onMaximizeClick = (): void => {
+        dis.dispatch({
+            action: 'video_fullscreen',
+            fullscreen: true,
+        });
+    };
+
     private onMicMuteClick = async (): Promise<void> => {
         const newVal = !this.state.micMuted;
         this.setState({ micMuted: await this.props.call.setMicrophoneMuted(newVal) });
@@ -617,6 +624,7 @@ export default class CallView extends React.Component<IProps, IState> {
                 onPipMouseDown={this.props.onMouseDownOnHeader}
                 pipMode={this.props.pipMode}
                 callRooms={[callRoom, secCallRoom]}
+                onMaximize={this.onMaximizeClick}
             />
             { contentView }
         </div>;

src/components/views/voip/CallView/CallViewHeader.tsx

@@ -19,48 +19,39 @@ import React from 'react';
 
 import { _t } from '../../../../languageHandler';
 import RoomAvatar from '../../avatars/RoomAvatar';
-import dis from '../../../../dispatcher/dispatcher';
-import { Action } from '../../../../dispatcher/actions';
 import AccessibleTooltipButton from '../../elements/AccessibleTooltipButton';
 
-interface CallViewHeaderProps {
-    pipMode: boolean;
-    callRooms?: Room[];
-    onPipMouseDown: (event: React.MouseEvent<Element, MouseEvent>) => void;
+interface CallControlsProps {
+    onExpand?: () => void;
+    onPin?: () => void;
+    onMaximize?: () => void;
 }
 
-const onFullscreenClick = () => {
-    dis.dispatch({
-        action: 'video_fullscreen',
-        fullscreen: true,
-    });
-};
-
-const onExpandClick = (roomId: string) => {
-    dis.dispatch({
-        action: Action.ViewRoom,
-        room_id: roomId,
-    });
-};
-
-type CallControlsProps = Pick<CallViewHeaderProps, 'pipMode'> & {
-    roomId: string;
-};
-const CallViewHeaderControls: React.FC<CallControlsProps> = ({ pipMode = false, roomId }) => {
+const CallViewHeaderControls: React.FC<CallControlsProps> = ({ onExpand, onPin, onMaximize }) => {
     return <div className="mx_CallViewHeader_controls">
-        { !pipMode && <AccessibleTooltipButton
+        { onMaximize && <AccessibleTooltipButton
             className="mx_CallViewHeader_button mx_CallViewHeader_button_fullscreen"
-            onClick={onFullscreenClick}
+            onClick={onMaximize}
             title={_t("Fill Screen")}
         /> }
-        { pipMode && <AccessibleTooltipButton
+        { onPin && <AccessibleTooltipButton
+            className="mx_CallViewHeader_button mx_CallViewHeader_button_pin"
+            onClick={onPin}
+            title={_t("Pin")}
+        /> }
+        { onExpand && <AccessibleTooltipButton
             className="mx_CallViewHeader_button mx_CallViewHeader_button_expand"
-            onClick={() => onExpandClick(roomId)}
+            onClick={onExpand}
             title={_t("Return to call")}
         /> }
     </div>;
 };
-const SecondaryCallInfo: React.FC<{ callRoom: Room }> = ({ callRoom }) => {
+
+interface ISecondaryCallInfoProps {
+    callRoom: Room;
+}
+
+const SecondaryCallInfo: React.FC<ISecondaryCallInfoProps> = ({ callRoom }) => {
     return <span className="mx_CallViewHeader_secondaryCallInfo">
         <RoomAvatar room={callRoom} height={16} width={16} />
         <span className="mx_CallView_secondaryCall_roomName">
@@ -69,19 +60,31 @@ const SecondaryCallInfo: React.FC<{ callRoom: Room }> = ({ callRoom }) => {
     </span>;
 };
 
+interface CallViewHeaderProps {
+    pipMode: boolean;
+    callRooms?: Room[];
+    onPipMouseDown: (event: React.MouseEvent<Element, MouseEvent>) => void;
+    onExpand?: () => void;
+    onPin?: () => void;
+    onMaximize?: () => void;
+}
+
 const CallViewHeader: React.FC<CallViewHeaderProps> = ({
     pipMode = false,
     callRooms = [],
     onPipMouseDown,
+    onExpand,
+    onPin,
+    onMaximize,
 }) => {
     const [callRoom, onHoldCallRoom] = callRooms;
-    const { roomId, name: callRoomName } = callRoom;
+    const callRoomName = callRoom.name;
 
     if (!pipMode) {
         return <div className="mx_CallViewHeader">
             <div className="mx_CallViewHeader_icon" />
             <span className="mx_CallViewHeader_text">{ _t("Call") }</span>
-            <CallViewHeaderControls roomId={roomId} pipMode={pipMode} />
+            <CallViewHeaderControls onMaximize={onMaximize} />
         </div>;
     }
     return (
@@ -94,7 +97,7 @@ const CallViewHeader: React.FC<CallViewHeaderProps> = ({
                 <div className="mx_CallViewHeader_roomName">{ callRoomName }</div>
                 { onHoldCallRoom && <SecondaryCallInfo callRoom={onHoldCallRoom} /> }
             </div>
-            <CallViewHeaderControls roomId={roomId} pipMode={pipMode} />
+            <CallViewHeaderControls onExpand={onExpand} onPin={onPin} onMaximize={onMaximize} />
         </div>
     );
 };

src/components/views/voip/PipView.tsx

@@ -19,6 +19,7 @@ import { CallEvent, CallState, MatrixCall } from 'matrix-js-sdk/src/webrtc/call'
 import { EventSubscription } from 'fbemitter';
 import { logger } from "matrix-js-sdk/src/logger";
 import classNames from 'classnames';
+import { Room } from "matrix-js-sdk/src/models/room";
 
 import CallView from "./CallView";
 import RoomViewStore from '../../../stores/RoomViewStore';
@@ -30,12 +31,13 @@ import { replaceableComponent } from "../../../utils/replaceableComponent";
 import PictureInPictureDragger from './PictureInPictureDragger';
 import dis from '../../../dispatcher/dispatcher';
 import { Action } from "../../../dispatcher/actions";
-import { WidgetLayoutStore } from '../../../stores/widgets/WidgetLayoutStore';
+import { Container, WidgetLayoutStore } from '../../../stores/widgets/WidgetLayoutStore';
 import CallViewHeader from './CallView/CallViewHeader';
 import ActiveWidgetStore, { ActiveWidgetStoreEvent } from '../../../stores/ActiveWidgetStore';
 import { UPDATE_EVENT } from '../../../stores/AsyncStore';
 import { RightPanelPhases } from '../../../stores/right-panel/RightPanelStorePhases';
 import RightPanelStore from '../../../stores/right-panel/RightPanelStore';
+import WidgetStore, { IApp } from "../../../stores/WidgetStore";
 
 const SHOW_CALL_IN_STATES = [
     CallState.Connected,
@@ -67,6 +69,17 @@ interface IState {
     moving: boolean;
 }
 
+const getRoomAndAppForWidget = (widgetId: string): [Room, IApp] => {
+    if (!widgetId) return;
+    const roomId = ActiveWidgetStore.instance.getRoomId(widgetId);
+    if (!roomId) return;
+
+    const room = MatrixClientPeg.get().getRoom(roomId);
+    const app = WidgetStore.instance.getApps(roomId).find((app) => app.id === widgetId);
+
+    return [room, app];
+};
+
 // Splits a list of calls into one 'primary' one and a list
 // (which should be a single element) of other calls.
 // The primary will be the one not on hold, or an arbitrary one
@@ -234,6 +247,37 @@ export default class PipView extends React.Component<IProps, IState> {
         }
     };
 
+    private onMaximize = (): void => {
+        const widgetId = this.state.persistentWidgetId;
+
+        if (this.state.showWidgetInPip && widgetId) {
+            const [room, app] = getRoomAndAppForWidget(widgetId);
+            WidgetLayoutStore.instance.moveToContainer(room, app, Container.Center);
+        } else {
+            dis.dispatch({
+                action: 'video_fullscreen',
+                fullscreen: true,
+            });
+        }
+    };
+
+    private onPin = (): void => {
+        if (!this.state.showWidgetInPip) return;
+
+        const [room, app] = getRoomAndAppForWidget(this.state.persistentWidgetId);
+        WidgetLayoutStore.instance.moveToContainer(room, app, Container.Top);
+    };
+
+    private onExpand = (): void => {
+        const widgetId = this.state.persistentWidgetId;
+        if (!widgetId || !this.state.showWidgetInPip) return;
+
+        dis.dispatch({
+            action: Action.ViewRoom,
+            room_id: ActiveWidgetStore.instance.getRoomId(widgetId),
+        });
+    };
+
     // Accepts a persistentWidgetId to be able to skip awaiting the setState for persistentWidgetId
     public updateShowWidgetInPip(persistentWidgetId = this.state.persistentWidgetId) {
         let userIsPartOfTheRoom = false;
@@ -285,13 +329,17 @@ export default class PipView extends React.Component<IProps, IState> {
             });
             const roomId = ActiveWidgetStore.instance.getRoomId(this.state.persistentWidgetId);
             const roomForWidget = MatrixClientPeg.get().getRoom(roomId);
+            const viewingCallRoom = this.state.viewedRoomId === roomId;
 
             pipContent = ({ onStartMoving, _onResize }) =>
                 <div className={pipViewClasses}>
                     <CallViewHeader
                         onPipMouseDown={(event) => { onStartMoving(event); this.onStartMoving.bind(this)(); }}
                         pipMode={pipMode}
                         callRooms={[roomForWidget]}
+                        onExpand={!viewingCallRoom && this.onExpand}
+                        onPin={viewingCallRoom && this.onPin}
+                        onMaximize={viewingCallRoom && this.onMaximize}
                     />
                     <PersistentApp
                         persistentWidgetId={this.state.persistentWidgetId}

src/i18n/strings/en_EN.json

@@ -1012,6 +1012,7 @@
     "More": "More",
     "Hangup": "Hangup",
     "Fill Screen": "Fill Screen",
+    "Pin": "Pin",
     "Return to call": "Return to call",
     "%(name)s on hold": "%(name)s on hold",
     "Call": "Call",
@@ -1095,7 +1096,6 @@
     "Anchor": "Anchor",
     "Headphones": "Headphones",
     "Folder": "Folder",
-    "Pin": "Pin",
     "Your server isn't responding to some <a>requests</a>.": "Your server isn't responding to some <a>requests</a>.",
     "Decline (%(counter)s)": "Decline (%(counter)s)",
     "Accept <policyLink /> to continue:": "Accept <policyLink /> to continue:",