launchpad-reviewers team mailing list archive

[Ines Almeida <mp+461260@xxxxxxxxxxxxxxxxxx>]

Ines Almeida has proposed merging ~ines-almeida/launchpad:frontpage-revamp-fix-demo-background-staging into launchpad:master.

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)


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ines-almeida/launchpad/+git/launchpad/+merge/461260

See https://qastaging.launchpad.net/
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ines-almeida/launchpad:frontpage-revamp-fix-demo-background-staging into launchpad:master.

diff --git a/lib/lp/app/templates/base-layout.pt b/lib/lp/app/templates/base-layout.pt
index 74b4f45..4ddb539 100644
--- a/lib/lp/app/templates/base-layout.pt
+++ b/lib/lp/app/templates/base-layout.pt
@@ -49,7 +49,7 @@
       use-macro="context/@@+base-layout-macros/launchpad-stylesheet-3-0" />
 
     <style tal:condition="is_demo" type="text/css">
-      html, body {background-image: url(/@@/demo) !important;}
+      html, body {background-image: url(/@@/demo) !important; background-repeat: repeat !important}
     </style>
 
     <tal:view condition="not: view/macro:is-page-contentless">