Closed Bug 1041685 Opened 6 years ago Closed 6 years ago
show hidpi icons in the mac global webrtc sharing indicator
I couldn't do this while working in bug 1037430 because of bug 899334 which is now fixed.
Hi Florian, can you mark this bug as either [qa+] or [qa-] for verification.
Assignee: nobody → florian
Status: NEW → ASSIGNED
Iteration: --- → 34.1
QA Whiteboard: [qa?]
QA Whiteboard: [qa?] → [qa+]
Based on bug 1037433 comment 17 and on attachment 8456098 [details] [diff] [review], this is how I think the patch should look like. But this doesn't work at all: (blank/empty) icons are displayed in the menubar. Will need to debug.
This time it actually works. The problem I had before was that the menu XUL node needs to be actually inserted in the document's DOM for the CSS to be applied. Should have been obvious :-/. This patch uses the icons from attachment 8464749 [details] (from bug 1037418). We tweaked them a few times with Philipp to have a reasonable size and alignment on both retina and non-retina screens.
Comment on attachment 8464800 [details] [diff] [review] Patch v2 Review of attachment 8464800 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/base/content/hiddenWindow.xul @@ +7,4 @@ > > #ifdef XP_MACOSX > <?xul-overlay href="chrome://browser/content/macBrowserOverlay.xul"?> > +<?xml-stylesheet href="chrome://browser/skin/webRTC-indicator.css" type="text/css"?> Part of me wants to call this "menubar-indicator", since it's more generic and could be used by other things. The other part of me doesn't care, given that it's taken us this long to find a need for it. :) We can always rename it later should the need arise.
Attachment #8464800 - Flags: review?(dolske) → review+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Comment on attachment 8464800 [details] [diff] [review] Patch v2 Approval Request Comment [Feature/regressing bug #]: bug 1037430 [User impact if declined]: blurry icons for Mac users with the menubar placed on a retina screen. [Describe test coverage new/current, TBPL]: currently in m-c, will be verified by QA. [Risks and why]: low [String/UUID change made/needed]: none
Attachment #8464800 - Flags: approval-mozilla-aurora?
Attachment #8464800 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
QA Contact: drno → florin.mezei
We have no Mac with Retina that we can use to verify this. As it seems that Nils will not handle this, Juan can you take care of it instead?
QA Contact: florin.mezei → jbecerra
As this is part of screensharing I'd like Nils to verify this.
Flags: needinfo?(jbecerra) → needinfo?(drno)
Not 100% sure what to look for exactly. But all three symbols (camera, microphone and screens) look ok on my retina display.
Marking as done.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa+] → [qa!]
You need to log in before you can comment on or make changes to this bug.