Closed Bug 872455 Opened 7 years ago Closed 7 years ago

icon in social toolbar button is mis-sized on retina display

Categories

(Firefox Graveyard :: SocialAPI, defect)

x86
macOS
defect
Not set

Tracking

(firefox22 unaffected, firefox23+ verified, firefox24 verified)

VERIFIED FIXED
Firefox 24
Tracking Status
firefox22 --- unaffected
firefox23 + verified
firefox24 --- verified

People

(Reporter: jfkthame, Assigned: Gijs)

References

Details

(Keywords: regression)

Attachments

(4 files)

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.
It is a regression, from bug 858321.
Keywords: regression
Attached patch PatchSplinter Review
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.
https://hg.mozilla.org/mozilla-central/rev/c6c467df9154
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?
Keywords: verifyme
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.
Status: RESOLVED → VERIFIED
Keywords: verifyme
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.