Closed
Bug 1131721
Opened 9 years ago
Closed 9 years ago
Editorial brands should have a background color on their details page
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
2015-02-24
People
(Reporter: krupa.mozbugs, Assigned: kngo)
References
()
Details
As per https://bug1131402.bugzilla.mozilla.org/attachment.cgi?id=8562216, Editorial brands should have a background color on their details page. This is currently missing. screenshot: https://www.dropbox.com/s/8h86ld7th35joq9/Screenshot%202015-02-10%2010.33.44.png?dl=0
Updated•9 years ago
|
Priority: -- → P3
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → kngo
Assignee | ||
Comment 1•9 years ago
|
||
https://github.com/mozilla/fireplace/pull/1023
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Target Milestone: --- → 2015-02-24
Comment 2•9 years ago
|
||
Verified as fixed on FF38(Win 7) in MP-dev. Postfix screencast http://screencast.com/t/AzQKqUf0i3t Closing bug.
Status: RESOLVED → VERIFIED
Comment 3•9 years ago
|
||
Sorry all, the grey icon ain't looking so hot. Here's my proposed plan: - Editorial icon in circle is 100% opacity - Curve is 75% - Background colour is 50%
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 4•9 years ago
|
||
That would affect the opacity tiers on other pages. What is our plan for those?
Comment 5•9 years ago
|
||
Seems like we should adjust the opacity if data-brand-landing is involved.
Assignee: kngo → dspasovski
Assignee | ||
Comment 6•9 years ago
|
||
This could probably be a separate bug at this point.
Assignee | ||
Updated•9 years ago
|
Assignee: dspasovski → kngo
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•9 years ago
|
||
Moving it back to verified state and doing Phil's comment in another bug.
You need to log in
before you can comment on or make changes to this bug.
Description
•