feat(b-tooltip, b-popover): allow global delay customization (bootstrap-vue#3426)

nfriend · tmorehouse · commit 2aaec76b6bc4 · 2019-05-31T14:19:49.000-03:00
diff --git a/src/components/popover/popover.js b/src/components/popover/popover.js
@@ -33,6 +33,10 @@ export const props = {
       return isArray(value) || arrayIncludes(['flip', 'clockwise', 'counterclockwise'], value)
     }
   },
+  delay: {
+    type: [Number, Object, String],
+    default: () => getComponentConfig(NAME, 'delay')
+  },
   boundary: {
     // String: scrollParent, window, or viewport
     // Element: element reference
diff --git a/src/components/tooltip/tooltip.js b/src/components/tooltip/tooltip.js
@@ -33,6 +33,10 @@ export default Vue.extend({
         return isArray(value) || arrayIncludes(['flip', 'clockwise', 'counterclockwise'], value)
       }
     },
+    delay: {
+      type: [Number, Object, String],
+      default: () => getComponentConfig(NAME, 'delay')
+    },
     boundary: {
       // String: scrollParent, window, or viewport
       // Element: element reference
diff --git a/src/directives/popover/popover.js b/src/directives/popover/popover.js
@@ -22,9 +22,11 @@ const validTriggers = {
 /* istanbul ignore next: not easy to test */
 const parseBindings = bindings => /* istanbul ignore next: not easy to test */ {
   // We start out with a basic config
+  const NAME = 'BPopover'
   let config = {
-    boundary: String(getComponentConfig('BPopover', 'boundary')),
-    boundaryPadding: parseInt(getComponentConfig('BPopover', 'boundaryPadding'), 10) || 0
+    delay: getComponentConfig(NAME, 'delay'),
+    boundary: String(getComponentConfig(NAME, 'boundary')),
+    boundaryPadding: parseInt(getComponentConfig(NAME, 'boundaryPadding'), 10) || 0
   }
 
   // Process bindings.value
diff --git a/src/directives/tooltip/tooltip.js b/src/directives/tooltip/tooltip.js
@@ -22,9 +22,11 @@ const validTriggers = {
 /* istanbul ignore next: not easy to test */
 const parseBindings = bindings => /* istanbul ignore next: not easy to test */ {
   // We start out with a basic config
+  const NAME = 'BTooltip'
   let config = {
-    boundary: String(getComponentConfig('BTooltip', 'boundary')),
-    boundaryPadding: parseInt(getComponentConfig('BTooltip', 'boundaryPadding'), 10) || 0
+    delay: getComponentConfig(NAME, 'delay'),
+    boundary: String(getComponentConfig(NAME, 'boundary')),
+    boundaryPadding: parseInt(getComponentConfig(NAME, 'boundaryPadding'), 10) || 0
   }
 
   // Process bindings.value
diff --git a/src/mixins/toolpop.js b/src/mixins/toolpop.js
@@ -42,10 +42,6 @@ export default {
       type: [String, Object, HTMLElement, Function]
       // default: undefined
     },
-    delay: {
-      type: [Number, Object, String],
-      default: 0
-    },
     offset: {
       type: [Number, String],
       default: 0
diff --git a/src/utils/config.js b/src/utils/config.js
@@ -144,10 +144,12 @@ const DEFAULTS = {
     role: null
   },
   BTooltip: {
+    delay: 0,
     boundary: 'scrollParent',
     boundaryPadding: 5
   },
   BPopover: {
+    delay: 0,
     boundary: 'scrollParent',
     boundaryPadding: 5
   }
