From 7742498fcd3f66ced2655c7b81bbd9b4d5410ca2 Mon Sep 17 00:00:00 2001
From: rnsloan <rnsloan1@gmail.com>
Date: Wed, 28 Oct 2015 03:01:25 +1100
Subject: [PATCH 1/2] move index page style block into stylus files

move interactive banner styles into new stylus file
add parameter to footer partial on homepage so it has a specific html class attribute and has specific css applied only on that page
---
 layouts/css/base.styl                         |  1 +
 layouts/css/page-modules/_footer.styl         |  3 ++
 .../css/page-modules/_interactiveBanner.styl  | 23 +++++++++++++
 layouts/index.hbs                             | 33 +------------------
 layouts/partials/footer.hbs                   |  6 +++-
 5 files changed, 33 insertions(+), 33 deletions(-)
 create mode 100644 layouts/css/page-modules/_interactiveBanner.styl

diff --git a/layouts/css/base.styl b/layouts/css/base.styl
index 61d4262f5c5f5..45763c2a704d1 100644
--- a/layouts/css/base.styl
+++ b/layouts/css/base.styl
@@ -85,6 +85,7 @@ pre
 @import 'page-modules/_foundation-members'
 @import 'page-modules/_in-the-news'
 @import 'page-modules/_download'
+@import 'page-modules/_interactiveBanner'
 @import 'page-modules/_scrollToTop'
 @import 'page-modules/_anchorLinks'
 
diff --git a/layouts/css/page-modules/_footer.styl b/layouts/css/page-modules/_footer.styl
index 92ca84b365e30..6b9328928fa67 100644
--- a/layouts/css/page-modules/_footer.styl
+++ b/layouts/css/page-modules/_footer.styl
@@ -12,6 +12,9 @@ footer
         background-color transparent
         text-decoration underline
 
+    &.no-margin-top
+        margin-top 0px
+
 
 .footer-nav
     display flex
diff --git a/layouts/css/page-modules/_interactiveBanner.styl b/layouts/css/page-modules/_interactiveBanner.styl
new file mode 100644
index 0000000000000..6af899b0deab1
--- /dev/null
+++ b/layouts/css/page-modules/_interactiveBanner.styl
@@ -0,0 +1,23 @@
+.interactive-banner
+    background #333 url("/static/images/nodejs-interactive-portland2015.jpg") 50% 0px no-repeat
+    background-size cover
+    height 300px
+    text-align center
+    width 100%
+    line-height 300px
+    margin-top 20px
+
+.interactive-banner img
+    display block
+    margin-left auto
+    margin-right auto
+    padding-top 80px
+    height 140px
+    -webkit-filter drop-shadow(5px 5px 5px #222)
+    filter drop-shadow(5px 5px 5px #222)
+
+@media screen and (max-width 550px)
+    .interactive-banner img
+        padding-top 100px
+        height auto
+        width 300px
diff --git a/layouts/index.hbs b/layouts/index.hbs
index 3ad1a774827a8..e983578b2074d 100644
--- a/layouts/index.hbs
+++ b/layouts/index.hbs
@@ -31,37 +31,6 @@
 
             </div>
 
-            <style>
-                div.interactive-banner {
-                    background: #333 url("/static/images/nodejs-interactive-portland2015.jpg") 50% 0px no-repeat;
-                    background-size: cover;
-                    height:300px;
-                    text-align:center;
-                    width:100%;
-                    line-height:300px;
-                    margin-top:20px;
-                }
-                div.interactive-banner img {
-                    display: block;
-                    margin-left: auto;
-                    margin-right: auto;
-                    padding-top:80px;
-                    height:140px;
-                    -webkit-filter: drop-shadow(5px 5px 5px #222);
-                    filter: drop-shadow(5px 5px 5px #222);
-                }
-                @media screen and (max-width: 550px) {
-                    div.interactive-banner img {
-                        padding-top:100px;
-                        height: auto;
-                        width: 300px;
-                    }
-                }
-                footer {
-                    margin-top:0px;
-                }
-            </style>
-
         </div>
     </div>
 
@@ -71,7 +40,7 @@
         </div>
     </a>
 
-    {{> footer }}
+    {{> footer className="no-margin-top" }}
     <script src="/static/js/download.js" async defer></script>
 </body>
 </html>
diff --git a/layouts/partials/footer.hbs b/layouts/partials/footer.hbs
index 7192d748ebea4..7e49dfba0726b 100644
--- a/layouts/partials/footer.hbs
+++ b/layouts/partials/footer.hbs
@@ -1,6 +1,10 @@
 <a href="#" id="scrollToTop">&uarr; <span>{{i18n.scrollToTop}}</span></a>
 
-<footer role="contentinfo">
+{{#if className}}
+    <footer class="{{className}}" role="contentinfo">
+{{else}}
+    <footer role="contentinfo">
+{{/if}}
 
     <div class="linuxfoundation-footer">
         <div class="container">

From 7610732d7194f7a80cf8df32a9c11211118775b0 Mon Sep 17 00:00:00 2001
From: rnsloan <rnsloan1@gmail.com>
Date: Wed, 28 Oct 2015 22:05:42 +1100
Subject: [PATCH 2/2] update handlebars conditional

---
 layouts/partials/footer.hbs | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/layouts/partials/footer.hbs b/layouts/partials/footer.hbs
index 7e49dfba0726b..0ff19e68972ed 100644
--- a/layouts/partials/footer.hbs
+++ b/layouts/partials/footer.hbs
@@ -1,10 +1,6 @@
 <a href="#" id="scrollToTop">&uarr; <span>{{i18n.scrollToTop}}</span></a>
 
-{{#if className}}
-    <footer class="{{className}}" role="contentinfo">
-{{else}}
-    <footer role="contentinfo">
-{{/if}}
+<footer {{#if className}}class="{{className}}"{{/if}} role="contentinfo">
 
     <div class="linuxfoundation-footer">
         <div class="container">