wave-ui 2.28.1 → 2.31.1

package/src/wave-ui/components/w-menu.vue

@@ -3,20 +3,20 @@
   slot(name="activator" :on="activatorEventHandlers")
   transition(:name="transitionName" appear)
     .w-menu(
-      v-if="custom && menuVisible"
-      ref="menu"
-      @click="hideOnMenuClick && closeMenu(true)"
+      v-if="custom && detachableVisible"
+      ref="detachable"
+      @click="hideOnMenuClick && close(true)"
       @mouseenter="showOnHover && (hoveringMenu = true)"
-      @mouseleave="showOnHover && ((hoveringMenu = false), closeMenu())"
+      @mouseleave="showOnHover && ((hoveringMenu = false), close())"
       :class="classes"
       :style="styles")
       slot
     w-card.w-menu(
-      v-else-if="menuVisible"
-      ref="menu"
-      @click.native="hideOnMenuClick && closeMenu(true)"
+      v-else-if="detachableVisible"
+      ref="detachable"
+      @click.native="hideOnMenuClick && close(true)"
       @mouseenter.native="showOnHover && (hoveringMenu = true)"
-      @mouseleave.native="showOnHover && ((hoveringMenu = false), closeMenu())"
+      @mouseleave.native="showOnHover && ((hoveringMenu = false), close())"
       :tile="tile"
       :title-class="titleClasses"
       :content-class="contentClasses"
@@ -32,12 +32,12 @@
   w-overlay(
     v-if="overlay"
     ref="overlay"
-    :model-value="menuVisible"
+    :model-value="detachableVisible"
     :persistent="persistent"
     :class="overlayClasses"
     v-bind="overlayProps"
     :z-index="(zIndex || 200) - 1"
-    @update:model-value="menuVisible = false")
+    @update:model-value="detachableVisible = false")
 </template>
 
 <script>
@@ -51,12 +51,13 @@
  */
 
 import { objectifyClasses } from '../utils/index'
-import { consoleWarn } from '../utils/console'
+import DetachableMixin from '../mixins/detachable'
 
 // const marginFromWindowSide = 4 // Amount of px from a window side, instead of overflowing.
 
 export default {
   name: 'w-menu',
+  mixins: [DetachableMixin],
 
   props: {
     modelValue: {}, // Show or hide.
@@ -73,98 +74,53 @@ export default {
     menuClass: { type: [String, Object, Array] },
     titleClass: { type: [String, Object, Array] },
     contentClass: { type: [String, Object, Array] },
-    // Position.
     arrow: { type: Boolean }, // The small triangle pointing toward the activator.
-    detachTo: { type: [String, Boolean, Object] },
-    fixed: { type: Boolean },
-    top: { type: Boolean },
-    bottom: { type: Boolean },
-    left: { type: Boolean },
-    right: { type: Boolean },
-    alignTop: { type: Boolean },
-    alignBottom: { type: Boolean },
-    alignLeft: { type: Boolean },
-    alignRight: { type: Boolean },
-    zIndex: { type: [Number, String, Boolean] },
     minWidth: { type: [Number, String] }, // can be like: `40`, `5em`, `activator`.
     overlay: { type: Boolean },
     overlayClass: { type: [String, Object, Array] },
     overlayProps: { type: Object }, // Allow passing down an object of props to the w-overlay component.
     persistent: { type: Boolean },
-    noPosition: { type: Boolean }
+    delay: { type: Number }
+    // Other props in the detachable mixin:
+    // detachTo, appendTo, fixed, top, bottom, left, right, alignTop, alignBottom, alignLeft,
+    // alignRight, noPosition, zIndex, activator.
   },
 
   emits: ['input', 'update:modelValue', 'open', 'close'],
 
   data: () => ({
-    menuVisible: false,
+    detachableVisible: false,
     hoveringActivator: false,
     hoveringMenu: false,
     // The menu computed top & left coordinates.
-    menuCoordinates: {
+    detachableCoords: {
       top: 0,
       left: 0
     },
-    activatorEl: null,
     activatorWidth: 0,
-    menuEl: null,
+    detachableEl: null,
     timeoutId: null
   }),
 
   computed: {
+    /**
+     * Other computed in the detachable mixin:
+     * - `appendToTarget`
+     * - `detachableParentEl`
+     * - `activatorEl`
+     * - `position`
+     * - `alignment`
+     **/
+
     transitionName () {
       return this.transition || 'scale-fade'
     },
 
-    // DOM element to attach menu to.
-    detachToTarget () {
-      let target = this.detachTo || '.w-app'
-      if (target === true) target = '.w-app'
-      else if (target && !['object', 'string'].includes(typeof target)) target = '.w-app'
-      else if (typeof target === 'object' && !target.nodeType) {
-        target = '.w-app'
-        consoleWarn('Invalid node provided in w-menu `detach-to`. Falling back to .w-app.', this)
-      }
-      if (typeof target === 'string') target = document.querySelector(target)
-
-      if (!target) {
-        consoleWarn(`Unable to locate ${this.detachTo ? `target ${this.detachTo}` : '.w-app'}`, this)
-        target = document.querySelector('.w-app')
-      }
-
-      return target
-    },
-
-    // DOM element that will receive the menu.
-    menuParentEl () {
-      return this.detachToTarget
-    },
-
-    position () {
-      return (
-        (this.top && 'top') ||
-        (this.bottom && 'bottom') ||
-        (this.left && 'left') ||
-        (this.right && 'right') ||
-        'bottom'
-      )
-    },
-
     menuMinWidth () {
       if (this.minWidth === 'activator') return this.activatorWidth ? `${this.activatorWidth}px` : 0
       else return isNaN(this.minWidth) ? this.minWidth : (this.minWidth ? `${this.minWidth}px` : 0)
     },
 
-    alignment () {
-      return (
-        ((this.top || this.bottom) && this.alignLeft && 'left') ||
-        ((this.top || this.bottom) && this.alignRight && 'right') ||
-        ((this.left || this.right) && this.alignTop && 'top') ||
-        ((this.left || this.right) && this.alignBottom && 'bottom') ||
-        ''
-      )
-    },
-
     menuClasses () {
       return objectifyClasses(this.menuClass)
     },
@@ -197,12 +153,12 @@ export default {
       }
     },
 
-    // The floatting menu styles.
+    // The floating menu styles.
     styles () {
       return {
         zIndex: this.zIndex || this.zIndex === 0 || (this.overlay && !this.zIndex && 200) || null,
-        top: (this.menuCoordinates.top && `${~~this.menuCoordinates.top}px`) || null,
-        left: (this.menuCoordinates.left && `${~~this.menuCoordinates.left}px`) || null,
+        top: (this.detachableCoords.top && `${~~this.detachableCoords.top}px`) || null,
+        left: (this.detachableCoords.left && `${~~this.detachableCoords.left}px`) || null,
         minWidth: (this.minWidth && this.menuMinWidth) || null,
         '--w-menu-bg-color': this.arrow && this.$waveui.colors[this.bgColor || 'white']
       }
@@ -213,31 +169,44 @@ export default {
 
       if (this.showOnHover) {
         handlers = {
-          focus: this.toggleMenu,
-          blur: this.toggleMenu,
+          focus: this.toggle,
+          blur: this.toggle,
           mouseenter: e => {
             this.hoveringActivator = true
-            this.openMenu(e)
+            this.open(e)
           },
           mouseleave: e => {
             this.hoveringActivator = false
             // Wait 10ms, the time to get the hoveringMenu updated on mouseenter on the menu.
             setTimeout(() => {
-              if (!this.hoveringMenu) this.closeMenu()
+              if (!this.hoveringMenu) this.close()
             }, 10)
           }
         }
-
-        if ('ontouchstart' in window) handlers.click = this.toggleMenu
+        // Check the window exists: SSR-proof.
+        if (typeof window !== 'undefined' && 'ontouchstart' in window) {
+          handlers.click = this.toggle
+        }
       }
-      else handlers = { click: this.toggleMenu }
+      else handlers = { click: this.toggle }
       return handlers
     }
   },
 
   methods: {
-    toggleMenu (e) {
-      let shouldShowMenu = this.menuVisible
+    /**
+     * Other methods in the `detachable` mixin:
+     * - `getActivatorCoordinates`
+     * - `computeDetachableCoords`
+     * - `onResize`
+     * - `onOutsideMousedown`
+     * - `insertInDOM`
+     * - `removeFromDOM`
+     **/
+
+    // ! \ This function uses the DOM - NO SSR (only trigger from beforeMount and later).
+    toggle (e) {
+      let shouldShowMenu = this.detachableVisible
       if ('ontouchstart' in window && this.showOnHover && e.type === 'click') {
         shouldShowMenu = !shouldShowMenu
       }
@@ -253,25 +222,29 @@ export default {
 
       this.timeoutId = clearTimeout(this.timeoutId)
 
-      if (shouldShowMenu) {
-        this.$emit('update:modelValue', (this.menuVisible = true))
-        this.$emit('input', true)
-        this.$emit('open')
-
-        this.openMenu(e)
-      }
-      else this.closeMenu()
+      if (shouldShowMenu) this.open(e)
+      else this.close()
     },
 
-    async openMenu (e) {
-      this.menuVisible = true
-      await this.insertMenu()
+    // ! \ This function uses the DOM - NO SSR (only trigger from beforeMount and later).
+    async open (e) {
+      // A tiny delay may help positioning the detachable correctly in case of multiple activators
+      // with different menu contents.
+      if (this.delay) await new Promise(resolve => setTimeout(resolve, this.delay))
+
+      this.detachableVisible = true
+
+      // If the activator is external, there might be multiple,
+      // so on open, the activator will be set to the event target.
+      if (this.activator) this.activatorEl = e.target
+
+      await this.insertInDOM()
 
       if (this.minWidth === 'activator') this.activatorWidth = this.activatorEl.offsetWidth
 
-      if (!this.noPosition) this.computeMenuPosition(e)
+      if (!this.noPosition) this.computeDetachableCoords(e)
 
-      // In `getCoordinates` accessing the menu computed styles takes a few ms (less than 10ms),
+      // In `getActivatorCoordinates` accessing the menu computed styles takes a few ms (less than 10ms),
       // if we don't postpone the Menu apparition it will start transition from a visible menu and
       // thus will not transition.
       this.timeoutId = setTimeout(() => {
@@ -289,199 +262,32 @@ export default {
      * - click of activator
      * - hover outside if showOnHover
      * - click inside menu if hideOnMenuClick.
+     * / ! \ This function uses the DOM - NO SSR (only trigger from beforeMount and later).
      *
      * @param {Boolean} force when showOnHover is set to true, hovering menu should keep it open.
      *                        But if hideOnMenuClick is also set to true, this should force close
      *                        even while hovering the menu.
      */
-    async closeMenu (force = false) {
+    async close (force = false) {
       // Might be already closed.
       // E.g. showOnHover & hideOnMenuClick: on click, force hide then mouseleave is also firing.
-      if (!this.menuVisible) return
+      if (!this.detachableVisible) return
 
       if (this.showOnHover && !force) {
         await new Promise(resolve => setTimeout(resolve, 10))
         if (this.showOnHover && (this.hoveringMenu || this.hoveringActivator)) return
       }
 
-      this.$emit('update:modelValue', (this.menuVisible = false))
+      this.$emit('update:modelValue', (this.detachableVisible = false))
       this.$emit('input', false)
       this.$emit('close')
       // Remove the mousedown listener if the menu got closed without a mousedown outside of the menu.
       document.removeEventListener('mousedown', this.onOutsideMousedown)
       window.removeEventListener('resize', this.onResize)
-    },
-
-    onOutsideMousedown (e) {
-      if (!this.menuEl.contains(e.target) && !this.activatorEl.contains(e.target)) {
-        this.$emit('update:modelValue', (this.menuVisible = false))
-        this.$emit('input', false)
-        this.$emit('close')
-        document.removeEventListener('mousedown', this.onOutsideMousedown)
-        window.removeEventListener('resize', this.onResize)
-      }
-    },
-
-    onResize () {
-      if (this.minWidth === 'activator') this.activatorWidth = this.activatorEl.offsetWidth
-      this.computeMenuPosition()
-    },
-
-    getCoordinates (e) {
-      // Get the activator coordinates relative to window.
-      const { top, left, width, height } = (e ? e.target : this.activatorEl).getBoundingClientRect()
-      let coords = { top, left, width, height }
-
-      // If absolute position, adjust top & left.
-      if (!this.fixed) {
-        const { top: targetTop, left: targetLeft } = this.menuParentEl.getBoundingClientRect()
-        const computedStyles = window.getComputedStyle(this.menuParentEl, null)
-        coords = {
-          ...coords,
-          top: top - targetTop + this.menuParentEl.scrollTop - parseInt(computedStyles.getPropertyValue('border-top-width')),
-          left: left - targetLeft + this.menuParentEl.scrollLeft - parseInt(computedStyles.getPropertyValue('border-left-width'))
-        }
-      }
-
-      return coords
-    },
-
-    computeMenuPosition (e) {
-      // Get the activator coordinates.
-      let { top, left, width, height } = this.getCoordinates(e)
-
-      // 1. First display the menu but hide it (So we can get its dimension).
-      // --------------------------------------------------
-      this.menuEl.style.visibility = 'hidden'
-      this.menuEl.style.display = 'flex'
-      const computedStyles = window.getComputedStyle(this.menuEl, null)
-
-      // 2. Position the menu top, left, right, bottom and apply chosen alignment.
-      // --------------------------------------------------
-      // Subtract half or full activator width or height and menu width or height according to the
-      // menu alignment.
-      // Note: the menu position relies on transform translate, the custom animation may override the
-      // css transform property so do without it i.e. no translateX(-50%), and recalculate top & left
-      // manually.
-      switch (this.position) {
-        case 'top': {
-          top -= this.menuEl.offsetHeight
-          if (this.alignRight) {
-            // left: 100% of activator.
-            left += width - this.menuEl.offsetWidth +
-                    parseInt(computedStyles.getPropertyValue('border-right-width'))
-          }
-          else if (!this.alignLeft) left += (width - this.menuEl.offsetWidth) / 2 // left: 50% of activator - half menu width.
-          break
-        }
-        case 'bottom': {
-          top += height
-          if (this.alignRight) {
-            // left: 100% of activator.
-            left += width - this.menuEl.offsetWidth +
-                    parseInt(computedStyles.getPropertyValue('border-right-width'))
-          }
-          else if (!this.alignLeft) left += (width - this.menuEl.offsetWidth) / 2 // left: 50% of activator - half menu width.
-          break
-        }
-        case 'left': {
-          left -= this.menuEl.offsetWidth
-          if (this.alignBottom) top += height - this.menuEl.offsetHeight
-          else if (!this.alignTop) top += (height - this.menuEl.offsetHeight) / 2 // top: 50% of activator - half menu height.
-          break
-        }
-        case 'right': {
-          left += width
-          if (this.alignBottom) {
-            top += height - this.menuEl.offsetHeight +
-                   parseInt(computedStyles.getPropertyValue('margin-top'))
-          }
-          else if (!this.alignTop) {
-            top += (height - this.menuEl.offsetHeight) / 2 + // top: 50% of activator - half menu height.
-                   parseInt(computedStyles.getPropertyValue('margin-top'))
-          }
-          break
-        }
-      }
-
-      // 3. Keep fully in viewport.
-      // @todo: do this.
-      // --------------------------------------------------
-      // if (this.position === 'top' && ((top - this.menuEl.offsetHeight) < 0)) {
-      //   const margin = - parseInt(computedStyles.getPropertyValue('margin-top'))
-      //   top -= top - this.menuEl.offsetHeight - margin - marginFromWindowSide
-      // }
-      // else if (this.position === 'left' && left - this.menuEl.offsetWidth < 0) {
-      //   const margin = - parseInt(computedStyles.getPropertyValue('margin-left'))
-      //   left -= left - this.menuEl.offsetWidth - margin - marginFromWindowSide
-      // }
-      // else if (this.position === 'right' && left + width + this.menuEl.offsetWidth > window.innerWidth) {
-      //   const margin = parseInt(computedStyles.getPropertyValue('margin-left'))
-      //   left -= left + width + this.menuEl.offsetWidth - window.innerWidth + margin + marginFromWindowSide
-      // }
-      // else if (this.position === 'bottom' && top + height + this.menuEl.offsetHeight > window.innerHeight) {
-      //   const margin = parseInt(computedStyles.getPropertyValue('margin-top'))
-      //   top -= top + height + this.menuEl.offsetHeight - window.innerHeight + margin + marginFromWindowSide
-      // }
-
-      // 4. Hide the menu again so the transition happens correctly.
-      // --------------------------------------------------
-      this.menuEl.style.visibility = null
-
-      // The menu coordinates are also recalculated while resizing window with open menu: keep the menu visible.
-      if (!this.menuVisible) this.menuEl.style.display = 'none'
-
-      this.menuCoordinates = { top, left }
-    },
-
-    insertMenu () {
-      return new Promise(resolve => {
-        this.$nextTick(() => {
-          this.menuEl = this.$refs.menu?.$el || this.$refs.menu
-
-          // Move the menu elsewhere in the DOM.
-          // wrapper.parentNode.insertBefore(this.menuEl, wrapper)
-          this.detachToTarget.appendChild(this.menuEl)
-          resolve()
-        })
-      })
-    },
-
-    removeMenu () {
-      if (this.menuEl && this.menuEl.parentNode) this.menuEl.remove()
-    }
-  },
-
-  mounted () {
-    const wrapper = this.$el
-    this.activatorEl = wrapper.firstElementChild
-    // Unwrap the activator element.
-    wrapper.parentNode.insertBefore(this.activatorEl, wrapper)
-
-    // Unwrap the overlay.
-    if (this.overlay) {
-      this.overlayEl = this.$refs.overlay?.$el
-      wrapper.parentNode.insertBefore(this.overlayEl, wrapper)
-    }
-
-    if (this.modelValue) this.toggleMenu({ type: 'click', target: this.activatorEl })
-  },
-
-  beforeUnmount () {
-    this.removeMenu()
-    if (this.overlay && this.overlayEl.parentNode) this.overlayEl.remove()
-    if (this.activatorEl && this.activatorEl.parentNode) this.activatorEl.remove()
-  },
-
-  watch: {
-    modelValue (bool) {
-      if (!!bool !== this.menuVisible) this.toggleMenu({ type: 'click', target: this.activatorEl })
-    },
-    detachTo () {
-      this.removeMenu()
-      this.insertMenu()
     }
   }
+
+  // watch, mounted & beforeDestroy hooks are set in the detachable.js mixin.
 }
 </script>
 

package/src/wave-ui/components/w-notification-manager.vue

@@ -11,8 +11,8 @@ transition-group(
       :key="notif._uid"
       v-model="notif._value"
       @close="notifManager.dismiss(notif._uid)"
-      v-bind="notif")
-      | {{ notif.message }}
+      v-bind="notifProps(notif)")
+      div(v-html="notif.message")
 </template>
 
 <script>
@@ -42,6 +42,13 @@ export default {
     }
   },
 
+  methods: {
+    notifProps (notif) {
+      const { _value, _uid, message, timeout, ...props } = notif
+      return props
+    }
+  },
+
   created () {
     this.notifManager = new NotificationManager()
   },

package/src/wave-ui/components/w-progress.vue

@@ -1,7 +1,10 @@
 <template lang="pug">
 .w-progress(:class="classes" :style="styles")
   //- Linear progress.
-  .w-progress__progress(v-if="!circle" :class="{ full: progressValue === 100 }" :style="`width: ${progressValue}%`")
+  .w-progress__progress(
+    v-if="!circle"
+    :class="{ full: progressValue === 100 }"
+    :style="`width: ${progressValue}%`")
 
   //- Circular progress.
   template(v-else)

package/src/wave-ui/components/w-rating.vue

@@ -3,7 +3,7 @@ component(
   ref="formEl"
   :is="formRegister ? 'w-form-element' : 'div'"
   v-bind="formRegister && { validators, inputValue: rating, disabled: isDisabled, readonly: isReadonly }"
-  :valid.sync="valid"
+  v-model:valid="valid"
   @reset="$emit('update:modelValue', rating = null);$emit('input', null)"
   :class="classes")
   input(:id="inputName" :name="inputName" type="hidden" :value="rating")

package/src/wave-ui/components/w-select.vue

@@ -21,7 +21,7 @@ component(
     v-model="showMenu"
     :menu-class="`w-select__menu ${menuClass || ''}`"
     transition="slide-fade-down"
-    :detach-to="(menuProps || {}).detachTo !== undefined ? (menuProps || {}).detachTo : '.w-app'"
+    :append-to="(menuProps || {}).appendTo !== undefined ? (menuProps || {}).appendTo : '.w-app'"
     align-left
     custom
     min-width="activator"
@@ -329,7 +329,7 @@ export default {
       setTimeout(() => {
         const itemIndex = this.inputValue.length ? this.inputValue[0].index : 0 // Real index starts at 0.
         // User visible index starts at 1.
-        this.$refs['w-list'].$el.querySelector(`#w-select-menu--${this._.uid}_item-${itemIndex + 1}`).focus()
+        this.$refs['w-list'].$el.querySelector(`#w-select-menu--${this._.uid}_item-${itemIndex + 1}`)?.focus()
       }, 100)
     },
 

package/src/wave-ui/components/w-spinner.vue

@@ -1,5 +1,5 @@
 <template lang="pug">
-.w-spinner(v-if="value || value === undefined" :class="classes" :style="styles")
+.w-spinner(v-if="modelValue || modelValue === undefined" :class="classes" :style="styles")
   span(v-if="isThreeDots")
 </template>
 
@@ -7,7 +7,7 @@
 export default {
   name: 'w-spinner',
   props: {
-    value: {},
+    modelValue: {},
     color: { type: String, default: 'primary' },
     xs: { type: Boolean },
     sm: { type: Boolean },

package/src/wave-ui/components/w-table.vue

@@ -313,7 +313,7 @@ export default {
           }
         )
 
-        // Keep external `expanded-rows.sync` updated.
+        // Keep external `expanded-rows.sync` (Vue 2) or v-model:expanded-rows (Vue 3) updated.
         this.$emit('update:expanded-rows', this.expandedRowsInternal)
       }
 
@@ -342,7 +342,7 @@ export default {
             }
           )
 
-          // Keep external `selected-rows.sync` updated.
+          // Keep external `selected-rows.sync` (Vue 2) or v-model:selected-rows (Vue 3) updated.
           this.$emit('update:selected-rows', this.selectedRowsInternal)
         }
       }

package/src/wave-ui/components/w-tag.vue

@@ -89,7 +89,6 @@ export default {
   background-color: rgba(255, 255, 255, 0.85);
   padding-left: 2 * $base-increment;
   padding-right: 2 * $base-increment;
-  font-size: round(0.85 * $base-font-size);
   cursor: default;
   user-select: none;
 
@@ -100,23 +99,35 @@ export default {
   &--tile {border-radius: initial;}
   &--shadow {box-shadow: $box-shadow;}
 
-  &.size--xs {font-size: round(0.7 * $base-font-size);}
+  &.size--xs {
+    $font-size: round(0.7 * $base-font-size);
+    font-size: $font-size;
+    line-height: $font-size + 2px;
+  }
   &.size--sm {
-    font-size: round(0.82 * $base-font-size);
+    $font-size: round(0.82 * $base-font-size);
+    font-size: $font-size;
+    line-height: $font-size + 2px;
     padding: round(0.25 * $base-increment) $base-increment;
   }
   &.size--md {
-    font-size: round(0.95 * $base-font-size);
+    $font-size: round(0.85 * $base-font-size);
+    font-size: $font-size;
+    line-height: $font-size + 4px;
     padding-top: round(0.25 * $base-increment);
     padding-bottom: round(0.25 * $base-increment);
   }
   &.size--lg {
-    font-size: round(1.1 * $base-font-size);
+    $font-size: round(1.1 * $base-font-size);
+    font-size: $font-size;
+    line-height: $font-size + 4px;
     padding-top: round(0.5 * $base-increment);
     padding-bottom: round(0.5 * $base-increment);
   }
   &.size--xl {
-    font-size: round(1.3 * $base-font-size);
+    $font-size: round(1.3 * $base-font-size);
+    font-size: $font-size;
+    line-height: $font-size + 4px;
     padding-top: round(1 * $base-increment);
     padding-bottom: round(1 * $base-increment);
   }

package/src/wave-ui/components/w-textarea.vue

@@ -116,7 +116,7 @@ export default {
       return listeners
     },
     hasValue () {
-      return this.inputValue
+      return this.inputValue || this.inputValue === 0
     },
     hasLabel () {
       return this.label || this.$slots.default