delayOnChange default changed to true

sir-dunxalot · May 22, 2016 · aca1a53 · aca1a53
1 parent c5a69b7
commit aca1a53
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -135,7 +135,7 @@ This does not affect the hiding of the tooltip. See also, [delayOnChange](#delay
 
 | Type    | Boolean |
 |---------|---------|
-| Default | false   |
+| Default | true    |
 
 Whether or not to enforce the delay even when the user transitions their cursor between multiple target elements with tooltips.
 

diff --git a/addon/components/tooltip-on-element.js b/addon/components/tooltip-on-element.js
@@ -30,7 +30,7 @@ export default EmberTetherComponent.extend({
   /* Options */
 
   delay: 0,
-  delayOnChange: false,
+  delayOnChange: true,
   duration: 0,
   effect: 'slide', // fade, slide, none
   event: 'hover', // hover, click, focus, none

diff --git a/tests/dummy/app/styles/app.scss b/tests/dummy/app/styles/app.scss
@@ -95,3 +95,7 @@ h3 {
 hr {
   @include rem(margin, 6, 0);
 }
+
+input {
+  @include rem(padding, 0.4, 0.6);
+}
diff --git a/tests/integration/components/delay-test.js b/tests/integration/components/delay-test.js
@@ -22,7 +22,7 @@ test('It animates with delay passed as a number', function(assert) {
     this.$().trigger('mouseover');
   });
 
-  /* Check the tooltip is shwon after the correct delay */
+  /* Check the tooltip is shown after the correct delay */
 
   run.later(() => {
     assertHide(assert, this);
-Original file line number
+Diff line change
@@ Expand Up / @@ -95,3 +95,7 @@ h3 { @@
     hr {
       @include rem(margin, 6, 0);
     }
+    input {
+      @include rem(padding, 0.4, 0.6);
+    }