Skip to content

Commit

Permalink
Screen sharing improvements
Browse files Browse the repository at this point in the history
  • Loading branch information
josh-signal committed Jun 1, 2021
1 parent 5a4fc5e commit 373ffe4
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 13 deletions.
12 changes: 4 additions & 8 deletions _locales/en/messages.json
Original file line number Diff line number Diff line change
Expand Up @@ -1464,23 +1464,19 @@
"description": "Shown as the title for the modal that requests screen recording permissions"
},
"calling__presenting--macos-permission-description": {
"message": "On an Apple Mac computer using macOS Catalina version 10.15 or later, Signal needs permission to access your computer's screen recording.",
"message": "Signal needs permission to access your computer's screen recording.",
"description": "Shown as the description for the modal that requests screen recording permissions"
},
"calling__presenting--permission-instruction-step1": {
"message": "Go to System Preferences and then click Security & Privacy.",
"message": "Go to System Preferences.",
"description": "Shown as the description for the modal that requests screen recording permissions"
},
"calling__presenting--permission-instruction-step2": {
"message": "Click Privacy.",
"message": "Click on the lock icon on the bottom left and enter your computer’s password.",
"description": "Shown as the description for the modal that requests screen recording permissions"
},
"calling__presenting--permission-instruction-step3": {
"message": "On the left, click Screen Recording.",
"description": "Shown as the description for the modal that requests screen recording permissions"
},
"calling__presenting--permission-instruction-step4": {
"message": "On the right, check the Signal box.",
"message": "On the right, check the box next to Signal. If you don’t see Signal in the list, click the + to add it.",
"description": "Shown as the description for the modal that requests screen recording permissions"
},
"calling__presenting--permission-open": {
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@
"redux-ts-utils": "3.2.2",
"reselect": "4.0.0",
"rimraf": "2.6.2",
"ringrtc": "https://github.com/signalapp/signal-ringrtc-node.git#17b22fc9d47605867608193202c54be06bce6f56",
"ringrtc": "https://github.com/signalapp/signal-ringrtc-node.git#d44ffdaa1edfdbdcdd2e002a278ed595fe4fa853",
"rotating-file-stream": "2.1.5",
"sanitize-filename": "1.6.3",
"sanitize.css": "11.0.0",
Expand Down
1 change: 0 additions & 1 deletion ts/components/NeedsScreenRecordingPermissionsModal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ export const NeedsScreenRecordingPermissionsModal = ({
<li>{i18n('calling__presenting--permission-instruction-step1')}</li>
<li>{i18n('calling__presenting--permission-instruction-step2')}</li>
<li>{i18n('calling__presenting--permission-instruction-step3')}</li>
<li>{i18n('calling__presenting--permission-instruction-step4')}</li>
</ol>
<Modal.ButtonFooter>
<Button
Expand Down
4 changes: 4 additions & 0 deletions ts/state/ducks/calling.ts
Original file line number Diff line number Diff line change
Expand Up @@ -631,6 +631,10 @@ function groupCallStateChange(
if (didSomeoneStartPresenting) {
callingTones.someonePresenting();
}

if (payload.connectionState === GroupCallConnectionState.NotConnected) {
ipcRenderer.send('close-screen-share-controller');
}
};
}

Expand Down
6 changes: 3 additions & 3 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -15728,9 +15728,9 @@ rimraf@^3.0.2, rimraf@~3.0.2:
dependencies:
glob "^7.1.3"

"ringrtc@https://github.com/signalapp/signal-ringrtc-node.git#17b22fc9d47605867608193202c54be06bce6f56":
version "2.10.1"
resolved "https://github.com/signalapp/signal-ringrtc-node.git#17b22fc9d47605867608193202c54be06bce6f56"
"ringrtc@https://github.com/signalapp/signal-ringrtc-node.git#d44ffdaa1edfdbdcdd2e002a278ed595fe4fa853":
version "2.10.3"
resolved "https://github.com/signalapp/signal-ringrtc-node.git#d44ffdaa1edfdbdcdd2e002a278ed595fe4fa853"

ripemd160@^2.0.0, ripemd160@^2.0.1:
version "2.0.1"
Expand Down

0 comments on commit 373ffe4

Please sign in to comment.