Merge ~ines-almeida/launchpad:frontpage-revamp-fix-demo-background-staging into launchpad:master

Proposed by Ines Almeida
Status: Merged
Approved by: Ines Almeida
Approved revision: 102554d8ac5297e593511891af461f66d82a63a2
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~ines-almeida/launchpad:frontpage-revamp-fix-demo-background-staging
Merge into: launchpad:master
Diff against target: 13 lines (+1/-1)
1 file modified
lib/lp/app/templates/base-layout.pt (+1/-1)
Reviewer Review Type Date Requested Status
Jürgen Gmach Approve
Review via email: mp+461260@code.launchpad.net

Commit message

ui: ensure 'demo' background is set for (qa)staging environments

After the changes to the homepage, it is no longer displaying the 'demo' background in qastaging due to a 'background-repeat: no-repeat' setting. This ensures that when we want to show this backgroud, this setting is set to 'repeat'.
The demo background makes it more obvious to users that this is a staging environment (not production)

Description of the change

To post a comment you must log in.
Revision history for this message
Jürgen Gmach (jugmac00) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lib/lp/app/templates/base-layout.pt b/lib/lp/app/templates/base-layout.pt
2index 74b4f45..4ddb539 100644
3--- a/lib/lp/app/templates/base-layout.pt
4+++ b/lib/lp/app/templates/base-layout.pt
5@@ -49,7 +49,7 @@
6 use-macro="context/@@+base-layout-macros/launchpad-stylesheet-3-0" />
7
8 <style tal:condition="is_demo" type="text/css">
9- html, body {background-image: url(/@@/demo) !important;}
10+ html, body {background-image: url(/@@/demo) !important; background-repeat: repeat !important}
11 </style>
12
13 <tal:view condition="not: view/macro:is-page-contentless">

Subscribers

People subscribed via source and target branches

to status/vote changes: