Add screen sharing controls blog post + web platform documentation #3869

beaufortfrancois · 2022-09-29T10:13:18Z

Live preview:

cc @eladalon1983

netlify · 2022-09-29T10:13:22Z

✅ Deploy Preview for developer-chrome-com ready!

Name	Link
🔨 Latest commit	`c8a2567`
🔍 Latest deploy log	https://app.netlify.com/sites/developer-chrome-com/deploys/6335a00cc35c52000981160d
😎 Deploy Preview	https://deploy-preview-3869--developer-chrome-com.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

eladalon1983 · 2022-09-29T11:32:30Z

site/en/blog/avoiding-oversharing-when-screen-sharing/index.md

+
+## Reordering the selection offered to the user {: #reordering }
+
+When [`getDisplayMedia()`] is called, users are presented with a media picker which allows them to share whichever display surface they would like any tab, window, or screen. Experience shows that the ordering of the offering affects the users’ selection. We are experimenting in Chrome&nbsp;107 on desktop with reordering the picker in order to nudge users towards tabs, which is the best option in the majority of cases.


This sentence becomes ungrammatical towards the end, with a single "like" trying to serve two purposes. (Any surface the users would like; options like...)

As agreed offline, I've removed any tab, window, or screen

eladalon1983 · 2022-09-29T11:34:52Z

site/en/blog/avoiding-oversharing-when-screen-sharing/index.md

+
+## Reordering the selection offered to the user {: #reordering }
+
+When [`getDisplayMedia()`] is called, users are presented with a media picker which allows them to share whichever display surface they would like any tab, window, or screen. Experience shows that the ordering of the offering affects the users’ selection. We are experimenting in Chrome&nbsp;107 on desktop with reordering the picker in order to nudge users towards tabs, which is the best option in the majority of cases.


"On desktop" might be redundant, as getDisplayMedia is only supported on desktop.

Good catch!

eladalon1983 · 2022-09-29T11:36:44Z

site/en/blog/avoiding-oversharing-when-screen-sharing/index.md

+
+### Privacy
+
+Previously, screens were offered as the most prominent option. This is the least privacy-preserving option, as users may end up exposing more personal information than they had originally intended. Such as:


Maybe we should s/prominent/default

eladalon1983 · 2022-09-29T11:39:13Z

site/en/blog/avoiding-oversharing-when-screen-sharing/index.md

+
+## What's next {: #future }
+
+[Conditional Focus] will allow the capturing web app to instruct the browser to either switch focus to the captured display surface, or to avoid such a focus change.


Shall we mention suppressLocalAudioPlayback as well, do you think?

eladalon1983 · 2022-09-29T11:40:26Z

site/en/docs/web-platform/screen-sharing-controls/index.md

+```js
+const stream = await navigator.mediaDevices.getDisplayMedia({
+  video: true,
+  displaySurface: "window", // Pre-select the "Window" pane in the media picker.


Max line length exceeded on my device.
https://screenshot.googleplex.com/3PWu7qAeyqaff3W

eladalon1983

Here, would using a numbered list perhaps ensure that people:

... would be less likely to miss the first bullet, which has no link?
... would an easier time remembering their position in the doc when jumping back and forth with the other doc?

Whether you accept this suggestion or not, this is now LGTM. :-)

beaufortfrancois · 2022-09-29T12:19:49Z

Whether you accept this suggestion or not, this is now LGTM. :-)

Fixed.

rachelandrew · 2022-09-29T13:15:11Z

@beaufortfrancois your glitch appears to be erroring in the docs.

rachelandrew · 2022-09-29T13:18:44Z

Whether you accept this suggestion or not, this is now LGTM. :-)

Fixed.

this should not be a numbered list, as per the style guide. https://developers.google.com/style/lists#types-of-lists please revert this to an unordered list as it is not a sequence of steps.

As a reminder, please make all changes in the Google doc and not on GitHub, and get any LGTMs there. We don't want to have to edit things twice.

beaufortfrancois · 2022-09-29T13:18:47Z

@beaufortfrancois your glitch appears to be erroring in the docs.

@rachelandrew It "works on my machine". Which error do you see?

rachelandrew · 2022-09-29T13:20:25Z

It's working now, the first two times I loaded the page I got an error saying there was a problem starting the glitch and the glitch showed the Glitch homepage. It's likely better just to link to it as Glitch is incredibly unreliable, I've been moving a lot of demos to CodePen when it's possible to do so.

beaufortfrancois · 2022-09-29T13:21:32Z

Whether you accept this suggestion or not, this is now LGTM. :-)

Fixed.

this should not be a numbered list, as per the style guide. https://developers.google.com/style/lists#types-of-lists please revert this to an unordered list as it is not a sequence of steps.

As a reminder, please make all changes in the Google doc and not on GitHub, and get any LGTMs there. We don't want to have to edit things twice.

I've reverted it.

beaufortfrancois · 2022-09-29T13:22:31Z

It's working now, the first two times I loaded the page I got an error saying there was a problem starting the glitch and the glitch showed the Glitch homepage. It's likely better just to link to it as Glitch is incredibly unreliable, I've been moving a lot of demos to CodePen when it's possible to do so.

Shall we use CodePen from now on? Is there official guidance for that? I wasn't aware of this.

rachelandrew · 2022-09-29T13:24:33Z

If CodePen works for the case then it's definitely a better option, it has rarely been unavailable. It isn't possible for every demo, but those where it is I'd suggest using it. We also have a team CodePen account so I can fork demos into that and then we are able to update them.

beaufortfrancois · 2022-09-29T13:26:03Z

If CodePen works for the case then it's definitely a better option, it has rarely been unavailable. It isn't possible for every demo, but those where it is I'd suggest using it. We also have a team CodePen account so I can fork demos into that and then we are able to update them.

Good to know! Moving forward I'll try to use CodePen then for embedded content on d.c.c. and web.dev then. Thank you!

rachelandrew

I just tweaked a heading, otherwise LGTM.

site/en/blog/avoiding-oversharing-when-screen-sharing/index.md

Co-authored-by: Rachel Andrew <rachelandrew@google.com>

beaufortfrancois · 2022-10-10T07:20:23Z

Note, the links at the bottom of the page are 404

displaySurface explainer
surfaceSwitching explainer
selfBrowserSurface explainer
systemAudio explainer

Thank you very much @guest271314!
I've just sent #3953 to address this issue

@eladalon1983 What do you think of the proposed note 4b00c01?

eladalon1983 · 2022-10-10T10:07:32Z

@eladalon1983 What do you think of the proposed note 4b00c01?

This looks like a clarification specific to getDisplayMedia(), and out of place in articles or blog posts centered around new controls. I'd rather not add it. The key to clear communication is focus.

eladalon1983 · 2022-10-10T12:44:47Z

Why withhold pertinent information?

Please refer back to the last sentence of my previous message.

eladalon1983 · 2022-10-10T12:54:44Z

@guest271314,

On the topic of API shape, if you refer to this slide, you'll see that I had proposed a different API shape, which I personally think would have been less confusing. However, working groups requires compromise, and systemAudio is the shape other members of the working group required. You're welcome to take this up with them.

On the topic of speechSynthesis, I see it as out-of-scope for this article. I don't think this is important for a significant enough chunk of this article's readership. That's my opinion.

eladalon1983 · 2022-10-10T12:56:09Z

@guest271314, I have a family as well as a full time job. Kindly condense your communication into a small number of short messages, or else I am afraid I will not have the time to read them and respond.

rachelandrew · 2022-10-10T13:05:28Z

I'm going to lock this thread, it seems as if there are a mixture of issues here and some of them seem to be about the spec, this isn't the best venue for that discussion.

If there are specific errors with the content, then they can be raised as a new issue, please raise them as individual issues.

Spec issues should be raised against the spec.

beaufortfrancois requested a review from rachelandrew September 29, 2022 10:13

WIP

3a76cfe

beaufortfrancois force-pushed the screen-sharing-controls branch from 3d7a8c0 to 3a76cfe Compare September 29, 2022 10:19

eladalon1983 reviewed Sep 29, 2022

View reviewed changes

Address @eladalon1983 feedback

6858c4b

eladalon1983 reviewed Sep 29, 2022

View reviewed changes

switch to numbered list

29cb1d5

Update screenshots

2341350

revert to unordered list

66abcbe

rachelandrew approved these changes Sep 29, 2022

View reviewed changes

site/en/blog/avoiding-oversharing-when-screen-sharing/index.md Outdated Show resolved Hide resolved

Update site/en/blog/avoiding-oversharing-when-screen-sharing/index.md

c8a2567

Co-authored-by: Rachel Andrew <rachelandrew@google.com>

beaufortfrancois merged commit 8e1d17d into main Sep 29, 2022

beaufortfrancois deleted the screen-sharing-controls branch September 29, 2022 13:45

This comment was marked as off-topic.

Sign in to view

beaufortfrancois mentioned this pull request Oct 10, 2022

Fix explainers URLs #3953

Merged

This comment was marked as off-topic.

Sign in to view

GoogleChrome locked as off-topic and limited conversation to collaborators Oct 10, 2022


		## Reordering the selection offered to the user {: #reordering }

		When [`getDisplayMedia()`] is called, users are presented with a media picker which allows them to share whichever display surface they would like any tab, window, or screen. Experience shows that the ordering of the offering affects the users’ selection. We are experimenting in Chrome 107 on desktop with reordering the picker in order to nudge users towards tabs, which is the best option in the majority of cases.


		### Privacy

		Previously, screens were offered as the most prominent option. This is the least privacy-preserving option, as users may end up exposing more personal information than they had originally intended. Such as:


		## What's next {: #future }

		[Conditional Focus] will allow the capturing web app to instruct the browser to either switch focus to the captured display surface, or to avoid such a focus change.

Add screen sharing controls blog post + web platform documentation #3869

Add screen sharing controls blog post + web platform documentation #3869

Conversation

beaufortfrancois commented Sep 29, 2022 • edited Loading

netlify bot commented Sep 29, 2022 • edited Loading

✅ Deploy Preview for developer-chrome-com ready!

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

eladalon1983 left a comment

Choose a reason for hiding this comment

beaufortfrancois commented Sep 29, 2022

rachelandrew commented Sep 29, 2022

rachelandrew commented Sep 29, 2022

beaufortfrancois commented Sep 29, 2022

rachelandrew commented Sep 29, 2022

beaufortfrancois commented Sep 29, 2022

beaufortfrancois commented Sep 29, 2022

rachelandrew commented Sep 29, 2022

beaufortfrancois commented Sep 29, 2022

rachelandrew left a comment

Choose a reason for hiding this comment

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

beaufortfrancois commented Oct 10, 2022

eladalon1983 commented Oct 10, 2022 • edited Loading

This comment was marked as off-topic.

eladalon1983 commented Oct 10, 2022

This comment was marked as off-topic.

eladalon1983 commented Oct 10, 2022

This comment was marked as off-topic.

eladalon1983 commented Oct 10, 2022

This comment was marked as off-topic.

This comment was marked as off-topic.

rachelandrew commented Oct 10, 2022

beaufortfrancois commented Sep 29, 2022 •

edited

Loading

netlify bot commented Sep 29, 2022 •

edited

Loading

eladalon1983 commented Oct 10, 2022 •

edited

Loading