Closed Bug 776431 Opened 13 years ago Closed 13 years ago

Share panel profile information doesn't appear in the first window, doesn't look great on Mac

Categories

(Firefox Graveyard :: SocialAPI, defect)

All
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 17

People

(Reporter: Gavin, Assigned: jaws)

References

Details

(Whiteboard: [fx16])

Attachments

(1 file, 1 obsolete file)

Combining two minor issues into one, to minimize aurora-uplift-tracking pain! - The share panel profile information block is only updated on window load, so if the provider sends the profile data after that, it doesn't get updated. In practice, this means it doesn't really ever show up in the first window. - When it does show up, on Mac, it doesn't look great, because the text is black-on-grey: http://cl.ly/image/081K3H3s1s3J
Whiteboard: [fx16]
Attached patch fix the updating issue (obsolete) — Splinter Review
This fixes the first issue.
The second issue looks to be because -moz-appearance:none on an OSX button still keeps the font color black. See here: https://mxr.mozilla.org/mozilla-central/source/browser/themes/pinstripe/browser.css#1270 Either hard-coding white, or some other solution will be needed here for OSX.
Attached patch PatchSplinter Review
This includes the fixes for both parts of this bug. The |color:#fff;| is the same that is used by @hudButton@ which is referenced by the #identity-popup-more-info-button so I think it will be OK to use here.
Attachment #644824 - Attachment is obsolete: true
Attachment #645608 - Flags: review?(gavin.sharp)
Attachment #645608 - Flags: review?(gavin.sharp) → review+
Flags: in-testsuite-
Target Milestone: --- → Firefox 17
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Depends on: 786207
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: