Clean up Reveal CSS

diff --git a/scss/components/_reveal.scss b/scss/components/_reveal.scss
index 253cee9cc446c331464a7dcb22e4565ba1825430..efffc199699fd15297cde6cf02b5df164bbd4dcb 100644 (file)
--- a/scss/components/_reveal.scss
+++ b/scss/components/_reveal.scss
@@ -66,14 +66,10 @@ $reveal-overlay-background: rgba($black, 0.45) !default;
   z-index: $reveal-zindex + 1;
   padding: $reveal-padding;
   border: $reveal-border;
-  // margin: $reveal-offset auto 0;
   background-color: $reveal-background;
   border-radius: $reveal-radius;
-
-  //chris's additions
   overflow-y: auto;
   max-height: 100vh;
-  //
 
   @include breakpoint(medium) {
     min-height: 0;
@@ -121,20 +117,11 @@ $reveal-overlay-background: rgba($black, 0.45) !default;
 }
 
 @mixin foundation-reveal {
-  // [TODO] Is this necessary?
   // scss-lint:disable QualifyingElement
   body.is-reveal-open {
     overflow: hidden;
   }
 
-  /* this little snippet causes the intended scroll blocking on iOS Safari,
-     which can still DIAF, because it breaks literally everything else */
-  // html.is-reveal-open, body.is-reveal-open {
-  //   overflow: hidden;
-  //   position: relative;
-  //   height: 100%;
-  // }
-
   // Overlay
   .reveal-overlay {
     @include reveal-overlay;
@@ -147,18 +134,10 @@ $reveal-overlay-background: rgba($black, 0.45) !default;
     position: absolute;
     overflow-y: auto;
 
-
-    // &.is-ios {
-    //   :last-child {
-    //     padding-bottom: 1em;
-    //   }
-    // }
-
-    /*** chris's addition ***/
     @include breakpoint(small) {
       width: 100vw;
     }
-    /************************/
+
     // Placeholder selector for medium-and-up modals
     // Prevents duplicate CSS when defining multiple Reveal sizes
     @include breakpoint(medium) {