898837 - Australis: Fix social-share-button CSS

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[:Gijs (he/him)]

When merging bug 886816 I noticed our social CSS is a little mixed up... Someone already noted this in bug 875488 comment 79 and bug 875488 comment 80, but I was too busy/lazy to followup at the time.

Note in particular:

It's missing from shared/toolbarbuttons.inc.css (http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/shared/toolbarbuttons.inc.css )

However:

http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/osx/browser.css#l471

(wrong id?)

http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/osx/browser.css#l903

(only in customization panel)

But then:

http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/osx/browser.css#l1873

And I don't know which of these wins because I don't know how to get this button to show up.

Because of this, I've left the button in our copy of @primaryToolbarButtons@ for now, because for Australis it *should* be using the sprite sheet, I believe. But this should probably be fixed properly.

(Marking P3 because I don't think this is seen by every user, and it might be OK for the moment but can use cleaning up? In any case, this might be P2 if the impact were bigger / brokenness is very bad right now on UX (which I can't tell))

Comment 1

[Shane Caraveo (:mixedpuppy)]

(In reply to :Gijs Kruitbosch from comment #0)

> http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/osx/
> browser.css#l471
> 
> (wrong id?)

seems wrong, but I haven't looked at the xul for australis

> http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/osx/
> browser.css#l903
> 
> (only in customization panel)
> 
> But then:
> 
> http://hg.mozilla.org/projects/ux/file/b0549e859d22/browser/themes/osx/
> browser.css#l1873

is this only from the merge?  If so, perhaps someone changed the share icon in australis already?

> And I don't know which of these wins because I don't know how to get this
> button to show up.

enable facebook or the sample provider at http://mixedpuppy.github.io/socialapi-demo/

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Shane, can you take this?

Comment 3

[Shane Caraveo (:mixedpuppy)]

Attachment: fix share css

this fixes the css for the button on osx and windows.  The css and images on linux do not look like they've been updated.

I have only tested this on osx non-retina, someone needs to validate windows/retina before this gets committed.

Comment 4

[:Gijs (he/him)]

Comment on attachment 796848 [details] [diff] [review]
fix share css

Review of attachment 796848 [details] [diff] [review]:
-----------------------------------------------------------------

The patch looks great to me, and it works on retina. I'll test Windows ASAP.

Comment 5

[:Gijs (he/him)]

This looks fine on Windows, too. If you have a tree, please land away and mark the whiteboard as [Australis:M9][Australis:P3][fixed-in-ux]. If not, no worries, I'll do it in ~9-10 hours. :-)

Comment 6

[:Gijs (he/him)]

Landed: https://hg.mozilla.org/projects/ux/rev/efd952ce34f8

Comment 7

[:Gijs (he/him)]

(In reply to :Gijs Kruitbosch from comment #6)
> Landed: https://hg.mozilla.org/projects/ux/rev/efd952ce34f8

Err, I meant https://hg.mozilla.org/projects/ux/rev/660f552f8160 . Oops!

Comment 9

[:Gijs (he/him)]

https://hg.mozilla.org/mozilla-central/rev/660f552f8160