quasar 2.19.3 → 2.20.0

package/src/composables/private.use-ratio/use-ratio.js

@@ -5,13 +5,14 @@ export const useRatioProps = {
 }
 
 export default function useRatio(props, naturalRatio) {
-  // return ratioStyle
   return computed(() => {
-    const ratio = Number(
-      props.ratio || (naturalRatio !== void 0 ? naturalRatio.value : void 0)
-    )
+    const rawValue = props.ratio || naturalRatio?.value
+    if (typeof rawValue === 'string' && rawValue.trim() === '') {
+      return null
+    }
 
-    return isNaN(ratio) !== true && ratio > 0
+    const ratio = Number(rawValue)
+    return Number.isFinite(ratio) && ratio > 0
       ? { paddingBottom: `${100 / ratio}%` }
       : null
   })

package/src/composables/private.use-ratio/use-ratio.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 
 import useRatio, { useRatioProps } from './use-ratio.js'
 

package/src/composables/private.use-refocus-target/use-refocus-target.js

@@ -1,12 +1,10 @@
-import { h, computed, ref } from 'vue'
+import { computed, h, ref } from 'vue'
 
 export default function useRefocusTarget(props, rootRef) {
   const refocusRef = ref(null)
 
   const refocusTargetEl = computed(() => {
-    if (props.disable === true) {
-      return null
-    }
+    if (props.disable) return null
 
     return h('span', {
       ref: refocusRef,

package/src/composables/private.use-router-link/use-router-link.js

@@ -25,11 +25,9 @@ function includesParams(outer, inner) {
       outerValue = outer[key]
 
     if (typeof innerValue === 'string') {
-      if (innerValue !== outerValue) {
-        return false
-      }
+      if (innerValue !== outerValue) return false
     } else if (
-      Array.isArray(outerValue) === false ||
+      !Array.isArray(outerValue) ||
       outerValue.length !== innerValue.length ||
       innerValue.some((value, i) => value !== outerValue[i])
     ) {
@@ -41,15 +39,15 @@ function includesParams(outer, inner) {
 }
 
 function isEquivalentArray(a, b) {
-  return Array.isArray(b) === true
+  return Array.isArray(b)
     ? a.length === b.length && a.every((value, i) => value === b[i])
     : a.length === 1 && a[0] === b
 }
 
 function isSameRouteLocationParamsValue(a, b) {
-  return Array.isArray(a) === true
+  return Array.isArray(a)
     ? isEquivalentArray(a, b)
-    : Array.isArray(b) === true
+    : Array.isArray(b)
       ? isEquivalentArray(b, a)
       : a === b
 }
@@ -60,7 +58,7 @@ function isSameRouteLocationParams(a, b) {
   }
 
   for (const key in a) {
-    if (isSameRouteLocationParamsValue(a[key], b[key]) === false) {
+    if (!isSameRouteLocationParamsValue(a[key], b[key])) {
       return false
     }
   }
@@ -106,53 +104,48 @@ export default function useRouterLink({
   const { props, proxy, emit } = vm
 
   const hasRouter = vmHasRouter(vm)
-  const hasHrefLink = computed(
-    () => props.disable !== true && props.href !== void 0
-  )
+  const hasHrefLink = computed(() => !props.disable && props.href !== void 0)
 
   // for perf reasons, we use minimum amount of runtime work
-  const hasRouterLinkProps =
-    useDisableForRouterLinkProps === true
-      ? computed(
-          () =>
-            hasRouter === true &&
-            props.disable !== true &&
-            hasHrefLink.value !== true &&
-            props.to !== void 0 &&
-            props.to !== null &&
-            props.to !== ''
-        )
-      : computed(
-          () =>
-            hasRouter === true &&
-            hasHrefLink.value !== true &&
-            props.to !== void 0 &&
-            props.to !== null &&
-            props.to !== ''
-        )
+  const hasRouterLinkProps = useDisableForRouterLinkProps
+    ? computed(
+        () =>
+          hasRouter &&
+          !props.disable &&
+          !hasHrefLink.value &&
+          props.to !== void 0 &&
+          props.to !== null &&
+          props.to !== ''
+      )
+    : computed(
+        () =>
+          hasRouter &&
+          !hasHrefLink.value &&
+          props.to !== void 0 &&
+          props.to !== null &&
+          props.to !== ''
+      )
 
   const resolvedLink = computed(() =>
-    hasRouterLinkProps.value === true ? getLink(props.to) : null
+    hasRouterLinkProps.value ? getLink(props.to) : null
   )
 
   const hasRouterLink = computed(() => resolvedLink.value !== null)
-  const hasLink = computed(
-    () => hasHrefLink.value === true || hasRouterLink.value === true
-  )
+  const hasLink = computed(() => hasHrefLink.value || hasRouterLink.value)
 
   const linkTag = computed(() =>
-    props.type === 'a' || hasLink.value === true
+    props.type === 'a' || hasLink.value
       ? 'a'
       : props.tag || fallbackTag || 'div'
   )
 
   const linkAttrs = computed(() =>
-    hasHrefLink.value === true
+    hasHrefLink.value
       ? {
           href: props.href,
           target: props.target
         }
-      : hasRouterLink.value === true
+      : hasRouterLink.value
         ? {
             href: resolvedLink.value.href,
             target: props.target
@@ -161,9 +154,7 @@ export default function useRouterLink({
   )
 
   const linkActiveIndex = computed(() => {
-    if (hasRouterLink.value === false) {
-      return -1
-    }
+    if (!hasRouterLink.value) return -1
 
     const { matched } = resolvedLink.value,
       { length } = matched,
@@ -183,9 +174,7 @@ export default function useRouterLink({
       isSameRouteRecord.bind(null, routeMatched)
     )
 
-    if (index !== -1) {
-      return index
-    }
+    if (index !== -1) return index
 
     // possible parent record
     const parentRecordPath = getOriginalPath(matched[length - 2])
@@ -198,7 +187,7 @@ export default function useRouterLink({
         // child of the same parent
         getOriginalPath(routeMatched) === parentRecordPath &&
         // avoid comparing the child with its parent
-        currentMatched[currentMatched.length - 1].path !== parentRecordPath
+        currentMatched.at(-1).path !== parentRecordPath
         ? currentMatched.findIndex(
             isSameRouteRecord.bind(null, matched[length - 2])
           )
@@ -208,25 +197,25 @@ export default function useRouterLink({
 
   const linkIsActive = computed(
     () =>
-      hasRouterLink.value === true &&
+      hasRouterLink.value &&
       linkActiveIndex.value !== -1 &&
       includesParams(proxy.$route.params, resolvedLink.value.params)
   )
 
   const linkIsExactActive = computed(
     () =>
-      linkIsActive.value === true &&
+      linkIsActive.value &&
       linkActiveIndex.value === proxy.$route.matched.length - 1 &&
       isSameRouteLocationParams(proxy.$route.params, resolvedLink.value.params)
   )
 
   const linkClass = computed(() =>
-    hasRouterLink.value === true
-      ? linkIsExactActive.value === true
+    hasRouterLink.value
+      ? linkIsExactActive.value
         ? ` ${props.exactActiveClass} ${props.activeClass}`
-        : props.exact === true
+        : props.exact
           ? ''
-          : linkIsActive.value === true
+          : linkIsActive.value
             ? ` ${props.activeClass}`
             : ''
       : ''
@@ -247,7 +236,7 @@ export default function useRouterLink({
     e,
     { returnRouterError, to = props.to, replace = props.replace } = {}
   ) {
-    if (props.disable === true) {
+    if (props.disable) {
       // ensure native navigation is prevented in all cases,
       // like when useDisableForRouterLinkProps === false (QRouteTab)
       e.preventDefault()
@@ -273,9 +262,9 @@ export default function useRouterLink({
     e.preventDefault()
 
     // then() can also return a "soft" router error (Vue Router behavior)
-    const promise = proxy.$router[replace === true ? 'replace' : 'push'](to)
+    const promise = proxy.$router[replace ? 'replace' : 'push'](to)
 
-    return returnRouterError === true
+    return returnRouterError
       ? promise
       : // else catching hard errors and also "soft" ones - then(err => ...)
         promise.then(() => {}).catch(() => {})
@@ -283,11 +272,11 @@ export default function useRouterLink({
 
   // warning! ensure that the component using it has 'click' included in its 'emits' definition prop
   function navigateOnClick(e) {
-    if (hasRouterLink.value === true) {
+    if (hasRouterLink.value) {
       const go = opts => navigateToRouterLink(e, opts)
 
       emit('click', e, go)
-      if (e.defaultPrevented !== true) go()
+      if (!e.defaultPrevented) go()
     } else {
       emit('click', e)
     }

package/src/composables/private.use-scroll-target/use-scroll-target.js

@@ -1,4 +1,4 @@
-import { ref, watch, onBeforeUnmount } from 'vue'
+import { onBeforeUnmount, ref, watch } from 'vue'
 
 import { listenOpts } from '../../utils/event/event.js'
 

package/src/composables/private.use-size/use-size.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 
 import useSize, { useSizeDefaults, useSizeProps } from './use-size.js'
 

package/src/composables/private.use-transition/use-transition.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 
 import useTransition, { useTransitionProps } from './use-transition.js'
 

package/src/composables/private.use-validate/use-validate.js

@@ -1,4 +1,4 @@
-import { ref, computed, watch, onBeforeUnmount, getCurrentInstance } from 'vue'
+import { computed, getCurrentInstance, onBeforeUnmount, ref, watch } from 'vue'
 
 import useFormChild from '../use-form/use-form-child.js'
 import { testPattern } from '../../utils/patterns/patterns.js'
@@ -45,17 +45,15 @@ export default function useValidate(focused, innerLoading) {
 
   const canDebounceValidate = computed(
     () =>
-      props.disable !== true &&
-      hasRules.value === true &&
+      !props.disable &&
+      hasRules.value &&
       // Should not have a validation in progress already;
       // It might mean that focus switched to submit btn and
       // QForm's submit() has been called already (ENTER key)
-      innerLoading.value === false
+      !innerLoading.value
   )
 
-  const hasError = computed(
-    () => props.error === true || innerError.value === true
-  )
+  const hasError = computed(() => props.error === true || innerError.value)
 
   const errorMessage = computed(() =>
     typeof props.errorMessage === 'string' && props.errorMessage.length !== 0
@@ -69,7 +67,7 @@ export default function useValidate(focused, innerLoading) {
       isDirtyModel.value = true
 
       if (
-        canDebounceValidate.value === true &&
+        canDebounceValidate.value &&
         // trigger validation if not using any kind of lazy-rules
         props.lazyRules === false
       ) {
@@ -81,8 +79,8 @@ export default function useValidate(focused, innerLoading) {
   function onRulesChange() {
     if (
       props.lazyRules !== 'ondemand' &&
-      canDebounceValidate.value === true &&
-      isDirtyModel.value === true
+      canDebounceValidate.value &&
+      isDirtyModel.value
     ) {
       debouncedValidate()
     }
@@ -91,7 +89,7 @@ export default function useValidate(focused, innerLoading) {
   watch(
     () => props.reactiveRules,
     val => {
-      if (val === true) {
+      if (val) {
         if (unwatchRules === void 0) {
           unwatchRules = watch(() => props.rules, onRulesChange, {
             immediate: true,
@@ -109,12 +107,9 @@ export default function useValidate(focused, innerLoading) {
   watch(() => props.lazyRules, onRulesChange)
 
   watch(focused, val => {
-    if (val === true) {
+    if (val) {
       isDirtyModel.value = true
-    } else if (
-      canDebounceValidate.value === true &&
-      props.lazyRules !== 'ondemand'
-    ) {
+    } else if (canDebounceValidate.value && props.lazyRules !== 'ondemand') {
       debouncedValidate()
     }
   })
@@ -135,23 +130,19 @@ export default function useValidate(focused, innerLoading) {
    *   - Promise (pending async validation)
    */
   function validate(val = props.modelValue) {
-    if (props.disable === true || hasRules.value === false) {
-      return true
-    }
+    if (props.disable || !hasRules.value) return true
 
     const index = ++validateIndex
+    const setDirty = innerLoading.value
+      ? () => {}
+      : () => {
+          isDirtyModel.value = true
+        }
 
-    const setDirty =
-      innerLoading.value !== true
-        ? () => {
-            isDirtyModel.value = true
-          }
-        : () => {}
-
-    const update = (err, msg) => {
-      if (err === true) setDirty()
+    const update = (hasErr, msg) => {
+      if (hasErr) setDirty()
 
-      innerError.value = err
+      innerError.value = hasErr
       innerErrorMessage.value = msg || null
       innerLoading.value = false
     }
@@ -185,11 +176,7 @@ export default function useValidate(focused, innerLoading) {
 
     return Promise.all(promises).then(
       res => {
-        if (
-          res === void 0 ||
-          Array.isArray(res) === false ||
-          res.length === 0
-        ) {
+        if (res === void 0 || !Array.isArray(res) || res.length === 0) {
           if (index === validateIndex) update(false)
           return true
         }
@@ -198,9 +185,9 @@ export default function useValidate(focused, innerLoading) {
         if (index === validateIndex) update(msg !== void 0, msg)
         return msg === void 0
       },
-      e => {
+      err => {
         if (index === validateIndex) {
-          console.error(e)
+          console.error(err)
           update(true)
         }
 

package/src/composables/use-dialog-plugin-component/use-dialog-plugin-component.js

@@ -1,4 +1,4 @@
-import { ref, getCurrentInstance } from 'vue'
+import { getCurrentInstance, ref } from 'vue'
 
 import getEmitsObject from '../../utils/private.get-emits-object/get-emits-object.js'
 

package/src/composables/use-form/private.use-form.js

@@ -1,4 +1,4 @@
-import { h, computed } from 'vue'
+import { computed, h } from 'vue'
 
 export const useFormProps = {
   name: String

package/src/composables/use-form/private.use-form.test.js

@@ -1,11 +1,11 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 import { computed } from 'vue'
 
 import {
-  useFormProps,
   useFormAttrs,
   useFormInject,
-  useFormInputNameAttr
+  useFormInputNameAttr,
+  useFormProps
 } from './private.use-form.js'
 
 describe('[useForm API]', () => {

package/src/composables/use-form/use-form-child.js

@@ -1,9 +1,9 @@
 import {
-  inject,
-  watch,
   getCurrentInstance,
+  inject,
+  onBeforeUnmount,
   onMounted,
-  onBeforeUnmount
+  watch
 } from 'vue'
 
 import { formKey } from '../../utils/private.symbols/symbols.js'
@@ -24,7 +24,7 @@ export default function useFormChild({
     watch(
       () => props.disable,
       val => {
-        if (val === true) {
+        if (val) {
           if (typeof resetValidation === 'function') resetValidation()
           $form.unbindComponent(proxy)
         } else {
@@ -35,14 +35,14 @@ export default function useFormChild({
 
     onMounted(() => {
       // register to parent QForm
-      if (props.disable !== true) $form.bindComponent(proxy)
+      if (!props.disable) $form.bindComponent(proxy)
     })
 
     onBeforeUnmount(() => {
       // un-register from parent QForm
-      if (props.disable !== true) $form.unbindComponent(proxy)
+      if (!props.disable) $form.unbindComponent(proxy)
     })
-  } else if (requiresQForm === true) {
+  } else if (requiresQForm) {
     console.error('Parent QForm not found on useFormChild()!')
   }
 }

package/src/composables/use-hydration/use-hydration.js

@@ -1,4 +1,4 @@
-import { ref, onMounted } from 'vue'
+import { onMounted, ref } from 'vue'
 
 // using it to manage SSR rendering with best performance
 import { isRuntimeSsrPreHydration } from '../../plugins/platform/Platform.js'
@@ -6,7 +6,7 @@ import { isRuntimeSsrPreHydration } from '../../plugins/platform/Platform.js'
 export default function useHydration() {
   const isHydrated = ref(!isRuntimeSsrPreHydration.value)
 
-  if (isHydrated.value === false) {
+  if (!isHydrated.value) {
     onMounted(() => {
       isHydrated.value = true
     })

package/src/composables/use-hydration/use-hydration.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 import { mount } from '@vue/test-utils'
 import { defineComponent } from 'vue'
 

package/src/composables/use-id/use-id.js

@@ -1,4 +1,4 @@
-import { ref, computed, watch, onMounted } from 'vue'
+import { computed, onMounted, ref, watch } from 'vue'
 
 import uid from '../../utils/uid/uid.js'
 
@@ -9,11 +9,7 @@ function parseValue(val) {
 }
 
 function getId(val, required) {
-  return val === void 0 || val === null
-    ? required === true
-      ? `f_${uid()}`
-      : null
-    : val
+  return val === void 0 || val === null ? (required ? `f_${uid()}` : null) : val
 }
 
 /**
@@ -24,10 +20,10 @@ function getId(val, required) {
  * avoid hydration errors.
  */
 export default function useId({ getValue, required = true } = {}) {
-  if (isRuntimeSsrPreHydration.value === true) {
+  if (isRuntimeSsrPreHydration.value) {
     const id = getValue !== void 0 ? ref(parseValue(getValue())) : ref(null)
 
-    if (required === true && id.value === null) {
+    if (required && id.value === null) {
       onMounted(() => {
         id.value = `f_${uid()}` // getId(null, true)
       })

package/src/composables/use-id/use-id.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 import { mount } from '@vue/test-utils'
 import { defineComponent } from 'vue'
 

package/src/composables/use-interval/use-interval.js

@@ -1,4 +1,4 @@
-import { onDeactivated, onBeforeUnmount, getCurrentInstance } from 'vue'
+import { getCurrentInstance, onBeforeUnmount, onDeactivated } from 'vue'
 
 import { vmIsDestroyed } from '../../utils/private.vm/vm.js'
 
@@ -28,7 +28,7 @@ export default function useInterval() {
     registerInterval(fn, delay) {
       removeInterval()
 
-      if (vmIsDestroyed(vm) === false) {
+      if (!vmIsDestroyed(vm)) {
         timer = setInterval(fn, delay)
       }
     }

package/src/composables/use-interval/use-interval.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect, vi, beforeEach, afterEach } from 'vitest'
+import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest'
 import { mount } from '@vue/test-utils'
 import { defineComponent } from 'vue'
 

package/src/composables/use-meta/use-meta.js

@@ -1,10 +1,10 @@
 import {
   computed,
-  watch,
   onActivated,
   onDeactivated,
   onUnmounted,
-  useSSRContext
+  useSSRContext,
+  watch
 } from 'vue'
 
 import { clientList, planClientUpdate } from '../../plugins/meta/Meta.js'
@@ -25,7 +25,7 @@ export default function useMeta(metaOptions) {
 
       watch(content, val => {
         meta.val = val
-        if (meta.active === true) planClientUpdate()
+        if (meta.active) planClientUpdate()
       })
     } else {
       meta.val = metaOptions

package/src/composables/use-quasar/use-quasar.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 import { mount } from '@vue/test-utils'
 import { defineComponent } from 'vue'
 

package/src/composables/use-render-cache/use-render-cache.js

@@ -18,7 +18,7 @@ export default function useRenderCache() {
     },
 
     hasCache(key) {
-      return Object.hasOwnProperty.call(cache, key)
+      return Object.hasOwn(cache, key)
     },
 
     clearCache(key) {

package/src/composables/use-render-cache/use-render-cache.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 
 import useRenderCache from './use-render-cache.js'
 

package/src/composables/use-split-attrs/use-split-attrs.js

@@ -1,4 +1,4 @@
-import { ref, onBeforeUpdate, getCurrentInstance } from 'vue'
+import { getCurrentInstance, onBeforeUpdate, ref } from 'vue'
 
 const listenerRE = /^on[A-Z]/
 
@@ -15,17 +15,13 @@ export default function useSplitAttrs() {
     const listeners = {}
 
     for (const key in attrs) {
-      if (
-        key !== 'class' &&
-        key !== 'style' &&
-        listenerRE.test(key) === false
-      ) {
+      if (key !== 'class' && key !== 'style' && !listenerRE.test(key)) {
         attributes[key] = attrs[key]
       }
     }
 
     for (const key in vnode.props) {
-      if (listenerRE.test(key) === true) {
+      if (listenerRE.test(key)) {
         listeners[key] = vnode.props[key]
       }
     }

package/src/composables/use-split-attrs/use-split-attrs.test.js

@@ -1,4 +1,4 @@
-import { describe, test, expect } from 'vitest'
+import { describe, expect, test } from 'vitest'
 import { mount } from '@vue/test-utils'
 import { defineComponent } from 'vue'
 

package/src/composables/use-tick/use-tick.js

@@ -1,8 +1,8 @@
 import {
+  getCurrentInstance,
   nextTick,
-  onDeactivated,
   onBeforeUnmount,
-  getCurrentInstance
+  onDeactivated
 } from 'vue'
 
 import { vmIsDestroyed } from '../../utils/private.vm/vm.js'
@@ -34,7 +34,7 @@ export default function useTick() {
         if (tickFn === fn) {
           // we also check if VM is destroyed, since if it
           // got to trigger one nextTick() we cannot stop it
-          if (vmIsDestroyed(vm) === false) tickFn()
+          if (!vmIsDestroyed(vm)) tickFn()
           tickFn = void 0
         }
       })