Merge pull request #5435 from guardian/container-spacing-pr-fixes

Container spacing pr fixes - follow up with #5422
guardian · Aug 7, 2014 · 20bcf16 · 20bcf16
2 parents dff759b + 9fc30be
commit 20bcf16
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/common/app/assets/stylesheets/module/facia/_items.scss b/common/app/assets/stylesheets/module/facia/_items.scss
@@ -105,7 +105,7 @@
     position: absolute;
     @include rem((
         min-height: gs-height(1),
-        min-width: gs-span(1),
+        min-width: gs-span(1)
     ));
     top: 0;
     right: 0;
@@ -156,7 +156,9 @@
         }
     }
     @include mq(wide) {
-        left: gs-span(15) + $gs-gutter;
+        @include rem((
+            left: gs-span(15) + $gs-gutter
+        ));
     }
 }
 .facia-slice {
@@ -417,6 +419,7 @@
     @include rem((
         margin-top: $gs-baseline*2
     ));
+
     .item--support-show & {
         display: block;
     }
@@ -579,13 +582,7 @@ $cta-icon-gap: 2px;
         background: transparent;
         color: $c-neutral1;
     }
-
 
-    @include mq($to: tablet) {
-        &[data-link-name=top-stories] .container__header {
-            display: none;
-        }
-    }
     @include mq(tablet) {
         @include rem((
             padding-top: $gs-baseline/2

diff --git a/common/app/views/fragments/containers/news.scala.html b/common/app/views/fragments/containers/news.scala.html
@@ -25,7 +25,9 @@
                     data-component="@FaciaComponentName(config, collection)">
                     <div class="facia-container__inner">
                         <div class="container__border tone-@style.tone tone-accent-border"></div>
-                        <div class="container__header">
+                        <div class="@RenderClasses(Map(
+                            ("container__header", true),
+                            ("hide-on-mobile", (FaciaComponentName(config, collection) == "top-stories"))))">
                             <h2 class="container__title">
                                 @fragments.containers.elements.dateOrTitle(collection, items)
                             </h2>