add $use-hover-media-query to control @media (hover:hover) usage

fd13e33a · Chris Rebert · eaaaa005 · fd13e33a · fd13e33a
Commit fd13e33a authored 10 years ago by Chris Rebert
Hide whitespace changes
Inline Side-by-side

Showing

with 46 additions and 13 deletions
+46 -13
--- a/scss/_variables.scss
+++ b/scss/_variables.scss
@@ -50,6 +50,8 @@ $enable-transitions:         true !default;
 $spacer:                     1rem !default;
 $border-width:               .075rem !default;

+$use-hover-media-query:      false !default;
+
 //== Typography
 //
 //## Font, line-height, and color for body text, headings, and more.

--- a/scss/mixins/_hover.scss
+++ b/scss/mixins/_hover.scss
 @mixin hover {
-  // See Media Queries Level 4: http://drafts.csswg.org/mediaqueries/#hover
-  // Currently shimmed by https://github.com/cvrebert/mq4-hover-hover-shim
-  @media (hover: hover) {
+  @if $use-hover-media-query {
+    // See Media Queries Level 4: http://drafts.csswg.org/mediaqueries/#hover
+    // Currently shimmed by https://github.com/cvrebert/mq4-hover-hover-shim
+    @media (hover: hover) {
+      &:hover { @content }
+    }
+  }
+  @else {
    &:hover { @content }
  }
 }

 @mixin hover-focus {
-  &:focus { @content }
-  @include hover { @content }
+  @if $use-hover-media-query {
+    &:focus { @content }
+    @include hover { @content }
+  }
+  @else {
+    &:focus,
+    &:hover {
+      @content
+    }
+  }
 }

 @mixin plain-hover-focus {
-  &,
-  &:focus {
-    @content
+  @if $use-hover-media-query {
+    &,
+    &:focus {
+      @content
+    }
+    @include hover { @content }
+  }
+  @else {
+    &,
+    &:focus,
+    &:hover {
+      @content
+    }
  }
-  @include hover { @content }
 }

 @mixin hover-focus-active {
-  &:focus,
-  &:active {
-    @content
+  @if $use-hover-media-query {
+    &:focus,
+    &:active {
+      @content
+    }
+    @include hover { @content }
+  }
+  @else {
+    &:focus,
+    &:active,
+    &:hover {
+      @content
+    }
  }
-  @include hover { @content }
 }