show campaign on trade lists

Turns out ~22% of our users initially land on a trade list.
We like to keep the campaign off the pages where space is at a
premium, so we try to whitelist it to major landing pages in order
to avoid accidentally creating a bad experience on some page :)
This commit is contained in:
Emi Matchu 2015-09-05 17:53:15 +00:00
parent a4079d2e0c
commit 7b17f70635
3 changed files with 7 additions and 2 deletions

View file

@ -3,6 +3,7 @@
@import "../partials/secondary_nav" @import "../partials/secondary_nav"
body.closet_hangers-index body.closet_hangers-index
+campaign-progress
+secondary-nav +secondary-nav
#title #title
@ -355,11 +356,11 @@ body.closet_hangers-index
display: block display: block
#toggle-help #toggle-help
display: inline display: inline-block
&.js &.js
#toggle-compare #toggle-compare
display: inline display: inline-block
#closet-hangers.comparing #closet-hangers.comparing
.object .object

View file

@ -65,6 +65,8 @@ class ClosetHangersController < ApplicationController
if @public_perspective && user_signed_in? if @public_perspective && user_signed_in?
current_user.assign_closeted_to_items!(items) current_user.assign_closeted_to_items!(items)
end end
@campaign = Campaign.current
end end
def petpage def petpage

View file

@ -1,3 +1,5 @@
- content_for :before_title, advertise_campaign_progress(@campaign)
- unless public_perspective? - unless public_perspective?
- title t('.title_for.you') - title t('.title_for.you')
- add_body_class 'current-user' - add_body_class 'current-user'