quasar 2.18.6 → 2.19.0

package/src/utils/open-url/open-url.js

@@ -3,60 +3,57 @@ import Platform from '../../plugins/platform/Platform.js'
 import { noop } from '../event/event.js'
 import { isNumber } from '../is/is.js'
 
-function parseFeatures (winFeatures) {
+function parseFeatures(winFeatures) {
   const cfg = Object.assign({ noopener: true }, winFeatures)
   const feat = []
   for (const key in cfg) {
-    const value = cfg[ key ]
+    const value = cfg[key]
     if (value === true) {
       feat.push(key)
-    }
-    else if (isNumber(value) || (typeof value === 'string' && value !== '')) {
+    } else if (isNumber(value) || (typeof value === 'string' && value !== '')) {
       feat.push(key + '=' + value)
     }
   }
   return feat.join(',')
 }
 
-function openWindow (url, reject, windowFeatures) {
+function openWindow(url, reject, windowFeatures) {
   let open = window.open
 
   if (Platform.is.cordova === true) {
     if (cordova?.InAppBrowser?.open !== void 0) {
       open = cordova.InAppBrowser.open
-    }
-    else if (navigator?.app !== void 0) {
+    } else if (navigator?.app !== void 0) {
       return navigator.app.loadUrl(url, {
         openExternal: true
       })
     }
   }
 
+  // When "noopener" is set (default), window.open() intentionally returns null
+  // even when the window was successfully opened. Per the HTML spec, "noreferrer"
+  // also implies "noopener". Only reject if neither is in effect, so that the
+  // null return can be treated as a blocked popup.
+  // Note: parseFeatures() normalizes values to booleans, so strict === true is safe.
+  const cfg = Object.assign({ noopener: true }, windowFeatures)
+  const hasNoopener = cfg.noopener === true || cfg.noreferrer === true
+
   const win = open(url, '_blank', parseFeatures(windowFeatures))
 
   if (win) {
-    Platform.is.desktop && win.focus()
+    if (Platform.is.desktop) win.focus()
     return win
-  }
-  else {
+  } else if (hasNoopener === false) {
     reject?.()
   }
 }
 
 export default (url, reject, windowFeatures) => {
-  if (
-    Platform.is.ios === true
-    && window.SafariViewController !== void 0
-  ) {
+  if (Platform.is.ios === true && window.SafariViewController !== void 0) {
     window.SafariViewController.isAvailable(available => {
       if (available) {
-        window.SafariViewController.show(
-          { url },
-          noop,
-          reject
-        )
-      }
-      else {
+        window.SafariViewController.show({ url }, noop, reject)
+      } else {
         openWindow(url, reject, windowFeatures)
       }
     })

package/src/utils/patterns/patterns.js

@@ -1,11 +1,13 @@
 // file referenced from docs
 
-const
-  hex = /^#[0-9a-fA-F]{3}([0-9a-fA-F]{3})?$/,
+const hex = /^#[0-9a-fA-F]{3}([0-9a-fA-F]{3})?$/,
   hexa = /^#[0-9a-fA-F]{4}([0-9a-fA-F]{4})?$/,
-  hexOrHexa = /^#([0-9a-fA-F]{3}|[0-9a-fA-F]{4}|[0-9a-fA-F]{6}|[0-9a-fA-F]{8})$/,
-  rgb = /^rgb\(((0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5]),){2}(0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5])\)$/,
-  rgba = /^rgba\(((0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5]),){2}(0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5]),(0|0\.[0-9]+[1-9]|0\.[1-9]+|1)\)$/
+  hexOrHexa =
+    /^#([0-9a-fA-F]{3}|[0-9a-fA-F]{4}|[0-9a-fA-F]{6}|[0-9a-fA-F]{8})$/,
+  rgb =
+    /^rgb\(((0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5]),){2}(0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5])\)$/,
+  rgba =
+    /^rgba\(((0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5]),){2}(0|[1-9][\d]?|1[\d]{0,2}|2[\d]?|2[0-4][\d]|25[0-5]),(0|0\.[0-9]+[1-9]|0\.[1-9]+|1)\)$/
 
 // Keep in sync with ui/types/api/validation.d.ts
 export const testPattern = {
@@ -19,8 +21,10 @@ export const testPattern = {
   // This is a basic helper validation.
   // For something more complex (like RFC 822) you should write and use your own rule.
   // We won't be accepting PRs to enhance the one below because of the reason above.
-  // eslint-disable-next-line
-  email: v => /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/.test(v),
+  email: v =>
+    /^(([^<>()[\]\\.,;:\s@"]+(\.[^<>()[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/.test(
+      v
+    ),
 
   hexColor: v => hex.test(v),
   hexaColor: v => hexa.test(v),

package/src/utils/patterns/patterns.test.js

@@ -1,3 +1,5 @@
+// oxlint-disable import/no-named-as-default-member
+
 import { describe, test, expect } from 'vitest'
 
 import patterns from './patterns.js'

package/src/utils/private.click-outside/click-outside.js

@@ -3,11 +3,10 @@ import { portalProxyList } from '../private.portal/portal.js'
 
 let timer = null
 
-const
-  { notPassiveCapture } = listenOpts,
+const { notPassiveCapture } = listenOpts,
   registeredList = []
 
-function globalHandler (evt) {
+function globalHandler(evt) {
   if (timer !== null) {
     clearTimeout(timer)
     timer = null
@@ -16,17 +15,19 @@ function globalHandler (evt) {
   const target = evt.target
 
   if (
-    target === void 0
-    || target.nodeType === 8
-    || target.classList.contains('no-pointer-events') === true
-  ) return
+    target === void 0 ||
+    target.nodeType === 8 ||
+    target.classList.contains('no-pointer-events') === true
+  ) {
+    return
+  }
 
   // check last portal vm if it's
   // a QDialog and not in seamless mode
   let portalIndex = portalProxyList.length - 1
 
   while (portalIndex >= 0) {
-    const proxy = portalProxyList[ portalIndex ].$
+    const proxy = portalProxyList[portalIndex].$
 
     // skip QTooltip portals
     if (proxy.type.name === 'QTooltip') {
@@ -44,33 +45,26 @@ function globalHandler (evt) {
   }
 
   for (let i = registeredList.length - 1; i >= 0; i--) {
-    const state = registeredList[ i ]
+    const state = registeredList[i]
 
     if (
-      (
-        state.anchorEl.value === null
-        || state.anchorEl.value.contains(target) === false
-      )
-      && (
-        target === document.body
-        || (
-          state.innerRef.value !== null
-          && state.innerRef.value.contains(target) === false
-        )
-      )
+      (state.anchorEl.value === null ||
+        state.anchorEl.value.contains(target) === false) &&
+      (target === document.body ||
+        (state.innerRef.value !== null &&
+          state.innerRef.value.contains(target) === false))
     ) {
       // mark the event as being processed by clickOutside
       // used to prevent refocus after menu close
       evt.qClickOutside = true
       state.onClickOutside(evt)
-    }
-    else {
+    } else {
       return
     }
   }
 }
 
-export function addClickOutside (clickOutsideProps) {
+export function addClickOutside(clickOutsideProps) {
   registeredList.push(clickOutsideProps)
 
   if (registeredList.length === 1) {
@@ -79,7 +73,7 @@ export function addClickOutside (clickOutsideProps) {
   }
 }
 
-export function removeClickOutside (clickOutsideProps) {
+export function removeClickOutside(clickOutsideProps) {
   const index = registeredList.findIndex(h => h === clickOutsideProps)
 
   if (index !== -1) {
@@ -91,8 +85,16 @@ export function removeClickOutside (clickOutsideProps) {
         timer = null
       }
 
-      document.removeEventListener('mousedown', globalHandler, notPassiveCapture)
-      document.removeEventListener('touchstart', globalHandler, notPassiveCapture)
+      document.removeEventListener(
+        'mousedown',
+        globalHandler,
+        notPassiveCapture
+      )
+      document.removeEventListener(
+        'touchstart',
+        globalHandler,
+        notPassiveCapture
+      )
     }
   }
 }

package/src/utils/private.config/instance-config.js

@@ -1,6 +1,6 @@
 export const globalConfig = {}
 export let globalConfigIsFrozen = false
 
-export function freezeGlobalConfig () {
+export function freezeGlobalConfig() {
   globalConfigIsFrozen = true
 }

package/src/utils/private.config/instance-config.test.js

@@ -1,14 +1,20 @@
 import { describe, test, expect } from 'vitest'
 import { mount, config } from '@vue/test-utils'
 
-import { globalConfig, globalConfigIsFrozen, freezeGlobalConfig } from './instance-config.js'
+import {
+  globalConfig,
+  globalConfigIsFrozen,
+  freezeGlobalConfig
+} from './instance-config.js'
 
 // We override Quasar install so we have a custom $q.config
-const quasarVuePlugin = config.global.plugins.find(entry => entry.name === 'Quasar')
+const quasarVuePlugin = config.global.plugins.find(
+  entry => entry.name === 'Quasar'
+)
 const { install } = quasarVuePlugin
 
-function mountWithConfig (config) {
-  quasarVuePlugin.install = app => install(app, { config })
+function mountWithConfig(mountConfig) {
+  quasarVuePlugin.install = app => install(app, { config: mountConfig })
   mount({ template: '<div />' })
 }
 
@@ -43,9 +49,7 @@ describe('[instanceConfig API]', () => {
   describe('[Functions]', () => {
     describe('[(function)freezeGlobalConfig]', () => {
       test('has correct return value', () => {
-        expect(
-          freezeGlobalConfig()
-        ).toBeUndefined()
+        expect(freezeGlobalConfig()).toBeUndefined()
 
         expect(globalConfigIsFrozen).toBe(true)
       })

package/src/utils/private.config/nodes.js

@@ -4,16 +4,13 @@ const nodesList = []
 const portalTypeList = []
 
 let portalIndex = 1
-let target = __QUASAR_SSR_SERVER__
-  ? void 0
-  : document.body
+let target = __QUASAR_SSR_SERVER__ ? void 0 : document.body
 
-export function createGlobalNode (id, portalType) {
+export function createGlobalNode(id, portalType) {
   const el = document.createElement('div')
 
-  el.id = portalType !== void 0
-    ? `q-portal--${ portalType }--${ portalIndex++ }`
-    : id
+  el.id =
+    portalType !== void 0 ? `q-portal--${portalType}--${portalIndex++}` : id
 
   if (globalConfig.globalNodes !== void 0) {
     const cls = globalConfig.globalNodes.class
@@ -29,7 +26,7 @@ export function createGlobalNode (id, portalType) {
   return el
 }
 
-export function removeGlobalNode (el) {
+export function removeGlobalNode(el) {
   const nodeIndex = nodesList.indexOf(el)
 
   nodesList.splice(nodeIndex, 1)
@@ -38,15 +35,18 @@ export function removeGlobalNode (el) {
   el.remove()
 }
 
-export function changeGlobalNodesTarget (newTarget) {
+export function changeGlobalNodesTarget(newTarget) {
   if (newTarget === target) return
 
   target = newTarget
 
   if (
-    target === document.body
+    target === document.body ||
     // or we have less than 2 dialogs:
-    || portalTypeList.reduce((acc, type) => (type === 'dialog' ? acc + 1 : acc), 0) < 2
+    portalTypeList.reduce(
+      (acc, type) => (type === 'dialog' ? acc + 1 : acc),
+      0
+    ) < 2
   ) {
     nodesList.forEach(node => {
       if (node.contains(target) === false) {
@@ -60,11 +60,11 @@ export function changeGlobalNodesTarget (newTarget) {
   const lastDialogIndex = portalTypeList.lastIndexOf('dialog')
 
   for (let i = 0; i < nodesList.length; i++) {
-    const el = nodesList[ i ]
+    const el = nodesList[i]
 
     if (
-      (i === lastDialogIndex || portalTypeList[ i ] !== 'dialog')
-      && el.contains(target) === false
+      (i === lastDialogIndex || portalTypeList[i] !== 'dialog') &&
+      el.contains(target) === false
     ) {
       target.appendChild(el)
     }

package/src/utils/private.config/nodes.test.js

@@ -1,6 +1,10 @@
 import { describe, test, expect, afterEach } from 'vitest'
 
-import { createGlobalNode, removeGlobalNode, changeGlobalNodesTarget } from './nodes.js'
+import {
+  createGlobalNode,
+  removeGlobalNode,
+  changeGlobalNodesTarget
+} from './nodes.js'
 import { globalConfig } from './instance-config.js'
 
 let el = null
@@ -18,62 +22,58 @@ describe('[nodes API]', () => {
   describe('[Functions]', () => {
     describe('[(function)createGlobalNode]', () => {
       test('createGlobalNode(id)', () => {
-        const el = createGlobalNode('first')
+        const element = createGlobalNode('first')
 
-        expect(el).toBeInstanceOf(Element)
-        expect(el.getAttribute('id')).toBe('first')
-        expect(el.getAttribute('class')).toBeNull()
-        expect(el.parentNode).toBe(document.body)
+        expect(element).toBeInstanceOf(Element)
+        expect(element.getAttribute('id')).toBe('first')
+        expect(element.getAttribute('class')).toBeNull()
+        expect(element.parentNode).toBe(document.body)
       })
 
       test('createGlobalNode(id) + globalNodes.class', () => {
         globalConfig.globalNodes = { class: 'some-class' }
 
-        const el = createGlobalNode('some-id')
+        const element = createGlobalNode('some-id')
 
-        expect(el).toBeInstanceOf(Element)
-        expect(el.getAttribute('id')).toBe('some-id')
-        expect(el.getAttribute('class')).toBe('some-class')
-        expect(el.parentNode).toBe(document.body)
+        expect(element).toBeInstanceOf(Element)
+        expect(element.getAttribute('id')).toBe('some-id')
+        expect(element.getAttribute('class')).toBe('some-class')
+        expect(element.parentNode).toBe(document.body)
       })
 
       test('createGlobalNode(id, portalType)', () => {
-        const el = createGlobalNode('port1', 'portType')
+        const element = createGlobalNode('port1', 'portType')
 
-        expect(el).toBeInstanceOf(Element)
-        expect(el.getAttribute('id')).toMatch(/^q-portal--portType--\d+$/)
-        expect(el.getAttribute('class')).toBeNull()
-        expect(el.parentNode).toBe(document.body)
+        expect(element).toBeInstanceOf(Element)
+        expect(element.getAttribute('id')).toMatch(/^q-portal--portType--\d+$/)
+        expect(element.getAttribute('class')).toBeNull()
+        expect(element.parentNode).toBe(document.body)
       })
 
       test('createGlobalNode(id, portalType) + globalNodes.class', () => {
         globalConfig.globalNodes = { class: 'port-class' }
 
-        const el = createGlobalNode('port1', 'portType')
+        const element = createGlobalNode('port1', 'portType')
 
-        expect(el).toBeInstanceOf(Element)
-        expect(el.getAttribute('id')).toMatch(/^q-portal--portType--\d+$/)
-        expect(el.getAttribute('class')).toBe('port-class')
-        expect(el.parentNode).toBe(document.body)
+        expect(element).toBeInstanceOf(Element)
+        expect(element.getAttribute('id')).toMatch(/^q-portal--portType--\d+$/)
+        expect(element.getAttribute('class')).toBe('port-class')
+        expect(element.parentNode).toBe(document.body)
       })
     })
 
     describe('[(function)removeGlobalNode]', () => {
       test('has correct return value', () => {
-        const el = createGlobalNode('rem')
+        const element = createGlobalNode('rem')
 
-        expect(
-          removeGlobalNode(el)
-        ).toBeUndefined()
+        expect(removeGlobalNode(element)).toBeUndefined()
 
-        expect(el.parentNode).toBeNull()
+        expect(element.parentNode).toBeNull()
       })
 
       test('does not error out when removing non-existing el', () => {
         const localEl = document.createElement('div')
-        expect(
-          removeGlobalNode(localEl)
-        ).toBeUndefined()
+        expect(removeGlobalNode(localEl)).toBeUndefined()
       })
     })
 
@@ -86,23 +86,17 @@ describe('[nodes API]', () => {
         ]
 
         elList.forEach(node => {
-          expect(
-            node.parentElement === document.body
-          ).toBe(true)
+          expect(node.parentElement === document.body).toBe(true)
         })
 
         const newTargetEl = document.createElement('div')
         newTargetEl.setAttribute('id', 'new-target')
         document.body.appendChild(newTargetEl)
 
-        expect(
-          changeGlobalNodesTarget(newTargetEl)
-        ).toBeUndefined()
+        expect(changeGlobalNodesTarget(newTargetEl)).toBeUndefined()
 
         elList.forEach(node => {
-          expect(
-            node.parentElement
-          ).toBe(newTargetEl)
+          expect(node.parentElement).toBe(newTargetEl)
         })
       })
     })

package/src/utils/private.create/create.js

@@ -1,8 +1,12 @@
 import { defineComponent, markRaw, reactive } from 'vue'
 import { injectProp } from '../private.inject-obj-prop/inject-obj-prop.js'
 
-export function createComponent (raw) { return markRaw(defineComponent(raw)) }
-export function createDirective (raw) { return markRaw(raw) }
+export function createComponent(raw) {
+  return markRaw(defineComponent(raw))
+}
+export function createDirective(raw) {
+  return markRaw(raw)
+}
 
 export const createReactivePlugin = __QUASAR_SSR_SERVER__
   ? (state, plugin) => {
@@ -16,8 +20,10 @@ export const createReactivePlugin = __QUASAR_SSR_SERVER__
         injectProp(
           plugin,
           name,
-          () => reactiveState[ name ],
-          val => { reactiveState[ name ] = val }
+          () => reactiveState[name],
+          val => {
+            reactiveState[name] = val
+          }
         )
       }
 

package/src/utils/private.create/create.test.js

@@ -1,7 +1,11 @@
 import { describe, test, expect } from 'vitest'
 import { isReactive } from 'vue'
 
-import { createComponent, createDirective, createReactivePlugin } from './create.js'
+import {
+  createComponent,
+  createDirective,
+  createReactivePlugin
+} from './create.js'
 
 describe('[create API]', () => {
   describe('[Functions]', () => {
@@ -44,13 +48,13 @@ describe('[create API]', () => {
           expect(descriptor.get).toBeTypeOf('function')
           expect(descriptor.set).toBeTypeOf('function')
 
-          plugin[ key ] = 'new-value'
-          expect(plugin[ key ]).toBe('new-value')
-          expect(state[ key ]).toBe('new-value')
+          plugin[key] = 'new-value'
+          expect(plugin[key]).toBe('new-value')
+          expect(state[key]).toBe('new-value')
 
-          state[ key ] = 'quasar'
-          expect(plugin[ key ]).toBe('quasar')
-          expect(state[ key ]).toBe('quasar')
+          state[key] = 'quasar'
+          expect(plugin[key]).toBe('quasar')
+          expect(state[key]).toBe('quasar')
         })
       })
     })