Closed Bug 872455 Opened 7 years ago Closed 7 years ago
icon in social toolbar button is mis-sized on retina display
31.30 KB, image/png
1.07 KB, patch
|Details | Diff | Splinter Review|
111.97 KB, image/png
70.35 KB, image/png
The problem is that http://mxr.mozilla.org/mozilla-central/source/browser/themes/osx/browser.css#3709 loads a "@2x" version of the icon (i.e. an image with 32x32 actual pixels) on a Retina display, but there's nothing in the style to constrain the size to 16x16 CSS pixels. So it gets displayed with its 32 pixels scaled to 32 CSS px, instead of providing "double resolution" in a 16 CSS px area.
Not clear why we should track this - it's not a regression and retina macs have been available for a while now.
Something like this? I removed the -moz-image-rect because the images only contain this icon, so it's unnecessary. I can add it back if that's desired, and add one for the 2x icon. Generally, as we're shipping this now, I'm not sure why this isn't in Toolbar.png?
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Attachment #752136 - Flags: review?(mhammond)
Yeah, I agree that this should be in Toolbar.png.
Comment on attachment 752136 [details] [diff] [review] Patch Review of attachment 752136 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, but I've no retina mac to test on - so r+ under the assumption you have tested it there. Would you mind also opening a bug to have the image added to Toolbar.png?
Attachment #752136 - Flags: review?(mhammond) → review+
Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/c6c467df9154 Filed bug 874773 for the Toolbar.png inclusion.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
In that case, tracking and looking for uplift nom.
Comment on attachment 752136 [details] [diff] [review] Patch [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 858321 User impact if declined: Users who enable and then disable a social provider will see an ugly (cut-off) icon in their toolbar. Testing completed (on m-c, etc.): Tested on m-c, looks much better there Risk to taking this patch (and alternatives if risky): zero to none, it's just CSS changes pertaining only to this button. String or IDL/UUID changes made by this patch: none
Attachment #752136 - Flags: approval-mozilla-aurora?
Attachment #752136 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Apologies for the delay; pushed to aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/9d3e8e88a722 .
Matt, can you please test this to verify it's fixed in Firefox 23?
QA Contact: mwobensmith
See attachments. Using Mac Retina: On FF23 2013-05-15, the button is blank. On FF23b4 2013-07-09, the button is blank when not engaged, and has a different image when toggled. Neither of these match the screenshot attached to this bug. More info welcomed, thank you.
(In reply to Matt Wobensmith from comment #13) > Created attachment 774237 [details] > Social toolbar button in 23b4 - inactive > > See attachments. > > Using Mac Retina: > > On FF23 2013-05-15, the button is blank. > > On FF23b4 2013-07-09, the button is blank when not engaged, and has a > different image when toggled. Neither of these match the screenshot attached > to this bug. > > More info welcomed, thank you. I think you're looking at the share button, which is not what this bug addressed. The share button still needs retina icons and we have bug 886816 for that. The button in question is the facebook icon you are looking at, however you need to turn off social from the menu in that button to see the cloud icon in the original image showing the problem.
Thank you Shane, that was the information that I needed. Confirmed bug FF23 2013-05-15. Confirmed fixed FF23b4 2013-07-09. Confirmed fixed FF24 2013-07-09.
You need to log in before you can comment on or make changes to this bug.