gUM - Camera/Microphone selection looks disabled

VERIFIED FIXED in Firefox 24

Status

()

defect
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: aaronmt, Assigned: wesj)

Tracking

(Blocks 1 bug, {polish})

Trunk
Firefox 24
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox24+ verified, fennec24+)

Details

(Whiteboard: [getUserMedia][android-gum-])

Attachments

(3 attachments)

Reporter

Description

6 years ago
Decided to try out the permission prompt in Nightly builds using a gUM test. Loaded http://simpl.info/getusermedia, received a prompt and noticed that both cameras look like they are disabled in the prompt. I can select either one fine.

--
Nightly (06/02)
HTC One (Android 4.1)
Reporter

Comment 1

6 years ago
Same on the S4, so it's not device specific.
Reporter

Updated

6 years ago
Summary: gUM - Camera selection looks disabled → gUM - Camera/Microphone selection looks disabled
Saw this on Galaxy Nexus as well. I believe the selection items are functional though. Just the coloring is off to send the perception that it's disabled.

Not blocking only because the selection items still remain functional. But this should be something we should fix (probably a low risk fix with good value).
Keywords: polish
Whiteboard: [getUserMedia][blocking-gum-][android-gum-]
Frontend bug, resetting assignee.
Assignee: gpascutto → nobody
Confirming both front and back cameras *appear* disabled, but are functional, when testing on a Nexus 4 :-)
Assignee: nobody → wjohnston
Whiteboard: [getUserMedia][blocking-gum-][android-gum-] → [getUserMedia][android-gum-]
Reporter

Comment 5

6 years ago
Note: This looks fine on Gingerbread.
tracking-fennec: ? → 24+
Assignee

Comment 6

6 years ago
Posted patch PatchSplinter Review
I have no idea why the default style for these is to look like this. I Found a little padding bug in this too, and removed some unnecessary cruft I left in.
Attachment #759310 - Flags: review?
Assignee

Updated

6 years ago
Attachment #759310 - Flags: review? → review?(sriram)
Comment on attachment 759310 [details] [diff] [review]
Patch

Looks good to me. Do we need to color change from #222 to #000?
Attachment #759310 - Flags: review?(sriram) → review+
https://hg.mozilla.org/mozilla-central/rev/14b4165dc0e2
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
Keywords: verifyme
Reporter

Updated

6 years ago
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.