bootstrap-vue · tmorehouse · May 3, 2019 · May 2, 2019 · May 2, 2019 · May 2, 2019
diff --git a/src/components/popover/popover.js b/src/components/popover/popover.js
@@ -1,9 +1,13 @@
 import Vue from '../../utils/vue'
 import PopOver from '../../utils/popover.class'
 import warn from '../../utils/warn'
+import { getComponentConfig } from '../../utils/config'
+import { HTMLElement } from '../../utils/safe-types'
 import normalizeSlotMixin from '../../mixins/normalize-slot'
 import toolpopMixin from '../../mixins/toolpop'
 
+const NAME = 'BPopover'
+
 export const props = {
   title: {
     type: String,
@@ -20,12 +24,22 @@ export const props = {
   placement: {
     type: String,
     default: 'right'
+  },
+  boundary: {
+    // String: scrollParent, window, or viewport
+    // Element: element reference
+    type: [String, HTMLElement],
+    default: () => getComponentConfig(NAME, 'boundary')
+  },
+  boundaryPadding: {
+    type: Number,
+    default: () => getComponentConfig(NAME, 'boundaryPadding')
   }
 }
 
 // @vue/component
 export default Vue.extend({
-  name: 'BPopover',
+  name: NAME,
   mixins: [toolpopMixin, normalizeSlotMixin],
   props,
   data() {

diff --git a/src/components/tooltip/tooltip.js b/src/components/tooltip/tooltip.js
@@ -1,12 +1,16 @@
 import Vue from '../../utils/vue'
 import ToolTip from '../../utils/tooltip.class'
 import warn from '../../utils/warn'
+import { getComponentConfig } from '../../utils/config'
+import { HTMLElement } from '../../utils/safe-types'
 import normalizeSlotMixin from '../../mixins/normalize-slot'
 import toolpopMixin from '../../mixins/toolpop'
 
+const NAME = 'BTooltip'
+
 // @vue/component
 export default Vue.extend({
-  name: 'BTooltip',
+  name: NAME,
   mixins: [toolpopMixin, normalizeSlotMixin],
   props: {
     title: {
@@ -20,6 +24,16 @@ export default Vue.extend({
     placement: {
       type: String,
       default: 'top'
+    },
+    boundary: {
+      // String: scrollParent, window, or viewport
+      // Element: element reference
+      type: [String, HTMLElement],
+      default: () => getComponentConfig(NAME, 'boundary')
+    },
+    boundaryPadding: {
+      type: Number,
+      default: () => getComponentConfig(NAME, 'boundaryPadding')
     }
   },
   data() {

diff --git a/src/directives/popover/popover.js b/src/directives/popover/popover.js
@@ -1,6 +1,7 @@
 import Popper from 'popper.js'
 import PopOver from '../../utils/popover.class'
 import warn from '../../utils/warn'
+import { getComponentConfig } from '../../utils/config'
 import { isBrowser } from '../../utils/env'
 import { isFunction, isObject, isString } from '../../utils/inspect'
 import { keys } from '../../utils/object'
@@ -20,8 +21,11 @@ const validTriggers = {
 // Arguments and modifiers take precedence over passed value config object
 /* istanbul ignore next: not easy to test */
 const parseBindings = bindings => /* istanbul ignore next: not easy to test */ {
-  // We start out with a blank config
-  let config = {}
+  // We start out with a basic config
+  let config = {
+    boundary: String(getComponentConfig('BPopover', 'boundary')),
+    boundaryPadding: parseInt(getComponentConfig('BPopover', 'boundaryPadding'), 10) || 0
+  }
 
   // Process bindings.value
   if (isString(bindings.value)) {
@@ -55,7 +59,7 @@ const parseBindings = bindings => /* istanbul ignore next: not easy to test */ {
     ) {
       // placement of popover
       config.placement = mod
-    } else if (/^(window|viewport)$/.test(mod)) {
+    } else if (/^(window|viewport|scrollParent)$/.test(mod)) {
       // Boundary of popover
       config.boundary = mod
     } else if (/^d\d+$/.test(mod)) {

diff --git a/src/directives/tooltip/tooltip.js b/src/directives/tooltip/tooltip.js
@@ -1,6 +1,7 @@
 import Popper from 'popper.js'
 import ToolTip from '../../utils/tooltip.class'
 import warn from '../../utils/warn'
+import { getComponentConfig } from '../../utils/config'
 import { isBrowser } from '../../utils/env'
 import { isFunction, isObject, isString } from '../../utils/inspect'
 import { keys } from '../../utils/object'
@@ -20,8 +21,11 @@ const validTriggers = {
 // Arguments and modifiers take precedence over passed value config object
 /* istanbul ignore next: not easy to test */
 const parseBindings = bindings => /* istanbul ignore next: not easy to test */ {
-  // We start out with a blank config
-  let config = {}
+  // We start out with a basic config
+  let config = {
+    boundary: String(getComponentConfig('BTooltip', 'boundary')),
+    boundaryPadding: parseInt(getComponentConfig('BTooltip', 'boundaryPadding'), 10) || 0
+  }
 
   // Process bindings.value
   if (isString(bindings.value)) {
@@ -55,7 +59,7 @@ const parseBindings = bindings => /* istanbul ignore next: not easy to test */ {
     ) {
       // Placement of tooltip
       config.placement = mod
-    } else if (/^(window|viewport)$/.test(mod)) {
+    } else if (/^(window|viewport|scrollParent)$/.test(mod)) {
       // Boundary of tooltip
       config.boundary = mod
     } else if (/^d\d+$/.test(mod)) {

diff --git a/src/mixins/toolpop.js b/src/mixins/toolpop.js
@@ -59,16 +59,6 @@ export default {
       type: String,
       default: null
     },
-    boundary: {
-      // String: scrollParent, window, or viewport
-      // Element: element reference
-      type: [String, HTMLElement],
-      default: 'scrollParent'
-    },
-    boundaryPadding: {
-      type: Number,
-      default: 5
-    },
     show: {
       type: Boolean,
       default: false

diff --git a/src/utils/config.js b/src/utils/config.js
@@ -142,6 +142,14 @@ const DEFAULTS = {
     ariaLive: 'polite',
     ariaAtomic: 'true',
     role: null
+  },
+  BTooltip: {
+    boundary: 'scrollParent',
+    boundaryPadding: 5
+  },
+  BPopover: {
+    boundary: 'scrollParent',
+    boundaryPadding: 5
   }
 }
 

diff --git a/src/utils/tooltip.class.js b/src/utils/tooltip.class.js
@@ -95,7 +95,8 @@ const Defaults = {
   container: false,
   fallbackPlacement: 'flip',
   callbacks: {},
-  boundary: 'scrollParent'
+  boundary: 'scrollParent',
+  boundaryPadding: 5
 }
 
 // Transition event names