Issue metadata
Sign in to add a comment
|
Microphone access icon is a camera |
||||||||||||||||||||||
Issue descriptionVersion: 52.0.2743.6/dev OS: OSX 10.6 The notification icon for when a page is using the microphone is a camera (see screenshot). My computer has no enabled cameras so I was surprised to see a camera enabled in the url bar! I think we're just using the same icon for all media, but this can be confusing and doesn't feel polished. The Polymer Material Design folks have a special icon for camera, mic, and camera+mic (icons:perm-camera-mic) that might work here: https://elements.polymer-project.org/elements/iron-icons?view=demo:demo/index.html&active=iron-icons
,
Nov 3 2016
,
Nov 29 2016
+emilyschechter@, do you think this is something we want to pursue? Marking its priority to 3 for now. Feel free to change.
,
Dec 14 2016
Assigning to Emily to determine if we want to push for a new UI here.
,
Jan 12 2017
Issue 602479 has been merged into this issue.
,
Jan 12 2017
More context: back when this was first added the internals apparently didn't plumb separate permissions here, but now they do. The page info bubble uses separate icons for these permissions already. We should be consistent with that.
,
Jan 20 2017
,
Apr 20 2017
,
Jun 15 2017
,
Jun 15 2017
Re #9: Whom does this need feedback from? It's easy to repro and understand what the issue is, right? https://www.chromium.org/getting-involved/bug-triage
,
Jun 15 2017
It appears to need feedback from Emily per comment #4.
,
Jul 14 2017
If there was something previously blocking determination of mic vs. camera as suggested in #6 then I don't think we need more input. This seems like a pretty obvious thing to do. I don't think it's a trivial change but it looks straightforward. I think this primarily consists of splitting the media models in chrome/browser/ui/content_settings into one for mic and one for camera. They're kind of calling out for that anyway, as they're full of if(mic) do this else if(camera) do that. estark, is there someone on your team that could try this?
,
Jul 14 2017
Address autocomplete appears to have clobbered one of the cc's. Restored.
,
Jul 16 2017
Marking available as I think this is still low priority for us.
,
Jul 17 2017
Let me put this on my fixit list. It is fairly trivial engineering-wise, one just needs to split the bubble class into two. The question is just what is desirable product-wise; whether to treat the permissions completely separately, or whether we still want to somehow acknowledge that camera/mic permissions can be requested by a website together.
,
Jul 18 2017
Thanks Martin :) Personally I think it's ok to treat the 2 permissions completely separately in page action just like we do in site settings and page info.
,
Jul 19 2017
I also think it's probably ideal to treat the two separately since we have different icons in page info/settings as Raymes says. However, I don't think this should be priority as PageActions IMO are not really understandable anyway, and we should work on replacing them with i.e. anchored toasts, vs. building more of them :)
,
Nov 10 2017
,
Feb 18 2018
,
Aug 1
,
Aug 16
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by pkasting@chromium.org
, Jun 7 2016