Update Sync tabs panel's list item colors

RESOLVED FIXED in Firefox 35

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: antlam, Assigned: nalexander, Mentored)

Tracking

(Blocks 1 bug)

unspecified
Firefox 36
x86
Android
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox35 fixed, firefox36 fixed, fennec35+)

Details

(Whiteboard: [lang=java])

Attachments

(3 attachments)

Noticed that the color of the device list item (expandable) was #F5F7F9. Could we update this to #F5F5F5 to be consistent with the other panels?
We should use the same color value that lucasr used in bug 1058660.

Although, synced tabs came along after the new toolbar landed, color consistency is part of this effort.
(In reply to :Margaret Leibovic from comment #1)
> We should use the same color value that lucasr used in bug 1058660.
> 
> Although, synced tabs came along after the new toolbar landed, color
> consistency is part of this effort.

My thoughts exactly. #F5F5F5 is the color that we're using for those "headers" in that bug :)
NI-ing Nick to get this on his radar :)
Flags: needinfo?(nalexander)
This shouldn't be too hard, I can mentor this if nalexander doesn't have time to work on it.
Mentor: margaret.leibovic
Whiteboard: [lang=java]

Updated

5 years ago
tracking-fennec: --- → ?
Blocks: remotetabsv2
Flags: needinfo?(nalexander)
Assignee: nobody → nalexander
tracking-fennec: ? → 35+
antlam: here are the new colors.  Verified using Developer Tools' eyedropper.
Attachment #8517003 - Flags: review?(alam)
Comment on attachment 8517003 [details]
Synced.Tabs.Client.Colors.png

Nice work - Thanks.
Attachment #8517003 - Flags: review?(alam) → review+
Comment on attachment 8516998 [details] [diff] [review]
Use regular home tab background color in Synced Tabs client.

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

This is trivial; marking as r=antlam.
Attachment #8516998 - Flags: review+
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/3c0e1e8fd446
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Since this is tracked for Fennec 35, do we want this to uplift?
Flags: qe-verify-
Comment on attachment 8516998 [details] [diff] [review]
Use regular home tab background color in Synced Tabs client.

Approval Request Comment
[Feature/regressing bug #]: none.

[User impact if declined]: visual inconsistency.

[Describe test coverage new/current, TBPL]: TBPL, local testing.

[Risks and why]: none.

[String/UUID change made/needed]: none.
Attachment #8516998 - Flags: approval-mozilla-aurora?
Attachment #8516998 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.