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)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 17
People
(Reporter: Gavin, Assigned: jaws)
References
Details
(Whiteboard: [fx16])
Attachments
(1 file, 1 obsolete file)
2.28 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
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
Reporter | ||
Updated•13 years ago
|
Whiteboard: [fx16]
Reporter | ||
Comment 1•13 years ago
|
||
This fixes the first issue.
Assignee | ||
Comment 2•13 years ago
|
||
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.
Assignee | ||
Comment 3•13 years ago
|
||
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)
Reporter | ||
Updated•13 years ago
|
Attachment #645608 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Flags: in-testsuite-
Target Milestone: --- → Firefox 17
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•