The default bug view has changed. See this FAQ.

Display name in share panel is white on gray on OS X

VERIFIED FIXED in Firefox 17

Status

()

Firefox
SocialAPI
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: jaws, Assigned: mixedpuppy)

Tracking

17 Branch
Firefox 18
All
Mac OS X
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox17 fixed)

Details

(Whiteboard: [Fx17][qa-])

Attachments

(1 attachment, 1 obsolete attachment)

See http://hg.mozilla.org/mozilla-central/file/8af2ff9c6018/browser/themes/pinstripe/browser.css#l1264

Now that bug 771284 landed, the share panel shows a white display name on a gray background. We should now switch this to use the standard color.
(Assignee)

Comment 1

5 years ago
Created attachment 657479 [details] [diff] [review]
recommend panel.patch

This fixes the font color as well as aligning the username with other text.
Attachment #657479 - Flags: review?(jaws)
(Assignee)

Updated

5 years ago
Assignee: nobody → mixedpuppy
Comment on attachment 657479 [details] [diff] [review]
recommend panel.patch

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

::: browser/themes/pinstripe/browser.css
@@ +1269,5 @@
>    font-weight: bold;
>  }
>  
>  #socialUserDisplayName > .button-box {
> +  -moz-margin-start: -5px;

Why is this necessary? Instead of using a negative margin here, is there a positive margin or padding elsewhere that can be removed?
(Assignee)

Comment 3

5 years ago
Created attachment 658194 [details] [diff] [review]
recommend panel.patch
Attachment #657479 - Attachment is obsolete: true
Attachment #657479 - Flags: review?(jaws)
Attachment #658194 - Flags: review?(jaws)
(Assignee)

Updated

5 years ago
Attachment #658194 - Flags: review?(jaws) → review?(felipc)
Attachment #658194 - Flags: review?(felipc) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
(Assignee)

Comment 4

5 years ago
Comment on attachment 658194 [details] [diff] [review]
recommend panel.patch

[Approval Request Comment]
issue affects aurora, minor change to get the right packing and color of the username
Attachment #658194 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/integration/mozilla-inbound/rev/e30a0c22e414
Keywords: checkin-needed
Attachment #658194 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/e30a0c22e414
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
https://hg.mozilla.org/releases/mozilla-aurora/rev/e253875d797f
status-firefox17: --- → fixed
Is this something that can/should be automated?
Flags: in-testsuite?
Whiteboard: [Fx17] → [Fx17][qa?]
(In reply to Anthony Hughes, Mozilla QA (:ashughes) from comment #8)
> Is this something that can/should be automated?

The cost/benefit here wouldn't be worth it.
Status: RESOLVED → VERIFIED
Thanks Jared, flagging [qa-].
Whiteboard: [Fx17][qa?] → [Fx17][qa-]
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.