Clean up & fix spacing

grommet · Aug 23, 2016 · baefae4 · baefae4
1 parent 45cce03
commit baefae4
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 9 deletions.
diff --git a/src/modules/Footer.js b/src/modules/Footer.js
@@ -60,7 +60,7 @@ export default class Footer extends Component {
     ));
 
     return (
-      <Accordion className="directory__accordion" icon={false}>
+      <Accordion className="directory__accordion">
         {panels}
       </Accordion>
     );

diff --git a/src/scss/_objects.footer.scss b/src/scss/_objects.footer.scss
@@ -2,12 +2,15 @@
 
   &.footer--dark {
     .grommetux-collapsible ul {
-      @include media-query(palm) {
-        background: #404040;
-      }
+      background: #404040;
     }
   }
 
+  .grommetux-collapsible ul {
+    padding: ($inuit-base-spacing-unit / 2) $inuit-base-spacing-unit;
+    background: #fff;
+  }
+
   &__logo {
     width: $inuit-base-spacing-unit * 5.5;
 
@@ -16,10 +19,15 @@
     }
   }
 
+  .grommetux-header {
+    padding: $inuit-base-spacing-unit;
+  }
+
   .grommetux-heading {
     padding: 0;
     font-size: $menu-font-size;
     font-weight: 600;
+    line-height: $inuit-base-spacing-unit * 1.25;
   }
 
   .directory__list {
@@ -29,6 +37,8 @@
     line-height: $inuit-base-spacing-unit * 1.25;
   }
 
+  .directory__accordion > li:last-child {
+    border-bottom: none;
   }
 
   ul {
@@ -38,12 +48,8 @@
     a:not(.grommetux-anchor) {
       text-decoration: none;
     }
+  }
 
-  .grommetux-collapsible ul {
-    @include media-query(palm) {
-      padding: ($inuit-base-spacing-unit / 2) $inuit-base-spacing-unit;
-      background: #fff;
-    }
 
   .footer__social {
     @include media-query(palm) {