quasar 2.19.2 → 2.20.0

package/src/directives/touch-pan/TouchPan.js

@@ -8,13 +8,13 @@ import {
 import {
   addEvt,
   cleanEvt,
-  position,
   leftClick,
+  noop,
+  position,
   prevent,
-  stop,
-  stopAndPrevent,
   preventDraggable,
-  noop
+  stop,
+  stopAndPrevent
 } from '../../utils/event/event.js'
 import { clearSelection } from '../../utils/private.selection/selection.js'
 import getSSRProps from '../../utils/private.noop-ssr-directive-transform/noop-ssr-directive-transform.js'
@@ -29,43 +29,43 @@ function getChanges(evt, ctx, isFinal) {
 
   const direction = ctx.direction
 
-  if (direction.horizontal === true && direction.vertical !== true) {
+  if (direction.horizontal && !direction.vertical) {
     dir = distX < 0 ? 'left' : 'right'
-  } else if (direction.horizontal !== true && direction.vertical === true) {
+  } else if (!direction.horizontal && direction.vertical) {
     dir = distY < 0 ? 'up' : 'down'
-  } else if (direction.up === true && distY < 0) {
+  } else if (direction.up && distY < 0) {
     dir = 'up'
     if (absX > absY) {
-      if (direction.left === true && distX < 0) {
+      if (direction.left && distX < 0) {
         dir = 'left'
-      } else if (direction.right === true && distX > 0) {
+      } else if (direction.right && distX > 0) {
         dir = 'right'
       }
     }
-  } else if (direction.down === true && distY > 0) {
+  } else if (direction.down && distY > 0) {
     dir = 'down'
     if (absX > absY) {
-      if (direction.left === true && distX < 0) {
+      if (direction.left && distX < 0) {
         dir = 'left'
-      } else if (direction.right === true && distX > 0) {
+      } else if (direction.right && distX > 0) {
         dir = 'right'
       }
     }
-  } else if (direction.left === true && distX < 0) {
+  } else if (direction.left && distX < 0) {
     dir = 'left'
     if (absX < absY) {
-      if (direction.up === true && distY < 0) {
+      if (direction.up && distY < 0) {
         dir = 'up'
-      } else if (direction.down === true && distY > 0) {
+      } else if (direction.down && distY > 0) {
         dir = 'down'
       }
     }
-  } else if (direction.right === true && distX > 0) {
+  } else if (direction.right && distX > 0) {
     dir = 'right'
     if (absX < absY) {
-      if (direction.up === true && distY < 0) {
+      if (direction.up && distY < 0) {
         dir = 'up'
-      } else if (direction.down === true && distY > 0) {
+      } else if (direction.down && distY > 0) {
         dir = 'down'
       }
     }
@@ -73,8 +73,8 @@ function getChanges(evt, ctx, isFinal) {
 
   let synthetic = false
 
-  if (dir === void 0 && isFinal === false) {
-    if (ctx.event.isFirst === true || ctx.event.lastDir === void 0) {
+  if (dir === void 0 && !isFinal) {
+    if (ctx.event.isFirst || ctx.event.lastDir === void 0) {
       return {}
     }
 
@@ -119,6 +119,10 @@ function getChanges(evt, ctx, isFinal) {
   }
 }
 
+function removeChildrenNoPointerEvents() {
+  document.body.classList.remove('no-pointer-events--children')
+}
+
 let uid = 0
 
 export default createDirective(
@@ -129,14 +133,14 @@ export default createDirective(
 
         beforeMount(el, { value, modifiers }) {
           // early return, we don't need to do anything
-          if (modifiers.mouse !== true && client.has.touch !== true) return
+          if (!modifiers.mouse && !client.has.touch) return
 
           function handleEvent(evt, mouseEvent) {
-            if (modifiers.mouse === true && mouseEvent === true) {
+            if (modifiers.mouse && mouseEvent) {
               stopAndPrevent(evt)
             } else {
-              if (modifiers.stop === true) stop(evt)
-              if (modifiers.prevent === true) prevent(evt)
+              if (modifiers.stop) stop(evt)
+              if (modifiers.prevent) prevent(evt)
             }
           }
 
@@ -174,32 +178,30 @@ export default createDirective(
             },
 
             start(evt, mouseEvent) {
-              if (client.is.firefox === true) preventDraggable(el, true)
+              if (client.is.firefox) preventDraggable(el, true)
               ctx.lastEvt = evt
 
               /*
                * Stop propagation so possible upper v-touch-pan don't catch this as well;
                * If we're not the target (based on modifiers), we'll re-emit the event later
                */
-              if (mouseEvent === true || modifiers.stop === true) {
+              if (mouseEvent || modifiers.stop) {
                 /*
                  * are we directly switching to detected state?
                  * clone event only otherwise
                  */
                 if (
-                  ctx.direction.all !== true &&
+                  !ctx.direction.all &&
                   // account for UMD too where modifiers will be lowercased to work
-                  (mouseEvent !== true ||
-                    (ctx.modifiers.mouseAllDir !== true &&
-                      ctx.modifiers.mousealldir !== true))
+                  (!mouseEvent ||
+                    (!ctx.modifiers.mouseAllDir && !ctx.modifiers.mousealldir))
                 ) {
-                  const clone =
-                    evt.type.indexOf('mouse') !== -1
-                      ? new MouseEvent(evt.type, evt)
-                      : new TouchEvent(evt.type, evt)
+                  const clone = evt.type.includes('mouse')
+                    ? new MouseEvent(evt.type, evt)
+                    : new TouchEvent(evt.type, evt)
 
-                  if (evt.defaultPrevented === true) prevent(clone)
-                  if (evt.cancelBubble === true) stop(clone)
+                  if (evt.defaultPrevented) prevent(clone)
+                  if (evt.cancelBubble) stop(clone)
 
                   Object.assign(clone, {
                     qKeyEvent: evt.qKeyEvent,
@@ -208,7 +210,8 @@ export default createDirective(
                     qClonedBy:
                       evt.qClonedBy === void 0
                         ? [ctx.uid]
-                        : evt.qClonedBy.concat(ctx.uid)
+                        : // oxlint-disable-next-line unicorn/prefer-spread
+                          evt.qClonedBy.concat(ctx.uid)
                   })
 
                   ctx.initialEvent = {
@@ -256,15 +259,12 @@ export default createDirective(
                 handleEvent(evt, isMouseEvt)
 
                 let cursor
-                if (
-                  modifiers.preserveCursor !== true &&
-                  modifiers.preservecursor !== true
-                ) {
+                if (!modifiers.preserveCursor && !modifiers.preservecursor) {
                   cursor = document.documentElement.style.cursor || ''
                   document.documentElement.style.cursor = 'grabbing'
                 }
 
-                if (isMouseEvt === true) {
+                if (isMouseEvt) {
                   document.body.classList.add('no-pointer-events--children')
                 }
                 document.body.classList.add('non-selectable')
@@ -279,20 +279,14 @@ export default createDirective(
 
                   document.body.classList.remove('non-selectable')
 
-                  if (isMouseEvt === true) {
-                    const remove = () => {
-                      document.body.classList.remove(
-                        'no-pointer-events--children'
-                      )
-                    }
-
+                  if (isMouseEvt) {
                     if (withDelayedFn !== void 0) {
                       setTimeout(() => {
-                        remove()
+                        removeChildrenNoPointerEvents()
                         withDelayedFn()
                       }, 50)
                     } else {
-                      remove()
+                      removeChildrenNoPointerEvents()
                     }
                   } else if (withDelayedFn !== void 0) {
                     withDelayedFn()
@@ -300,8 +294,8 @@ export default createDirective(
                 }
               }
 
-              if (ctx.event.detected === true) {
-                if (ctx.event.isFirst !== true) {
+              if (ctx.event.detected) {
+                if (!ctx.event.isFirst) {
                   handleEvent(evt, ctx.event.mouse)
                 }
 
@@ -311,17 +305,13 @@ export default createDirective(
                   if (ctx.handler(payload) === false) {
                     ctx.end(evt)
                   } else {
-                    if (
-                      ctx.styleCleanup === void 0 &&
-                      ctx.event.isFirst === true
-                    ) {
+                    if (ctx.styleCleanup === void 0 && ctx.event.isFirst) {
                       start()
                     }
 
                     ctx.event.lastX = payload.position.left
                     ctx.event.lastY = payload.position.top
-                    ctx.event.lastDir =
-                      synthetic === true ? void 0 : payload.direction
+                    ctx.event.lastDir = synthetic ? void 0 : payload.direction
                     ctx.event.isFirst = false
                   }
                 }
@@ -330,11 +320,10 @@ export default createDirective(
               }
 
               if (
-                ctx.direction.all === true ||
+                ctx.direction.all ||
                 // account for UMD too where modifiers will be lowercased to work
-                (isMouseEvt === true &&
-                  (ctx.modifiers.mouseAllDir === true ||
-                    ctx.modifiers.mousealldir === true))
+                (isMouseEvt &&
+                  (ctx.modifiers.mouseAllDir || ctx.modifiers.mousealldir))
               ) {
                 start()
                 ctx.event.detected = true
@@ -347,12 +336,12 @@ export default createDirective(
 
               if (absX !== absY) {
                 if (
-                  (ctx.direction.horizontal === true && absX > absY) ||
-                  (ctx.direction.vertical === true && absX < absY) ||
-                  (ctx.direction.up === true && absX < absY && distY < 0) ||
-                  (ctx.direction.down === true && absX < absY && distY > 0) ||
-                  (ctx.direction.left === true && absX > absY && distX < 0) ||
-                  (ctx.direction.right === true && absX > absY && distX > 0)
+                  (ctx.direction.horizontal && absX > absY) ||
+                  (ctx.direction.vertical && absX < absY) ||
+                  (ctx.direction.up && absX < absY && distY < 0) ||
+                  (ctx.direction.down && absX < absY && distY > 0) ||
+                  (ctx.direction.left && absX > absY && distX < 0) ||
+                  (ctx.direction.right && absX > absY && distX > 0)
                 ) {
                   ctx.event.detected = true
                   ctx.move(evt)
@@ -366,19 +355,16 @@ export default createDirective(
               if (ctx.event === void 0) return
 
               cleanEvt(ctx, 'temp')
-              if (client.is.firefox === true) preventDraggable(el, false)
+              if (client.is.firefox) preventDraggable(el, false)
 
-              if (abort === true) {
+              if (abort) {
                 ctx.styleCleanup?.()
 
-                if (
-                  ctx.event.detected !== true &&
-                  ctx.initialEvent !== void 0
-                ) {
+                if (!ctx.event.detected && ctx.initialEvent !== void 0) {
                   ctx.initialEvent.target.dispatchEvent(ctx.initialEvent.event)
                 }
-              } else if (ctx.event.detected === true) {
-                if (ctx.event.isFirst === true) {
+              } else if (ctx.event.detected) {
+                if (ctx.event.isFirst) {
                   ctx.handler(
                     getChanges(evt === void 0 ? ctx.lastEvt : evt, ctx).payload
                   )
@@ -408,25 +394,23 @@ export default createDirective(
 
           el.__qtouchpan = ctx
 
-          if (modifiers.mouse === true) {
+          if (modifiers.mouse) {
             // account for UMD too where modifiers will be lowercased to work
             const capture =
-              modifiers.mouseCapture === true || modifiers.mousecapture === true
-                ? 'Capture'
-                : ''
+              modifiers.mouseCapture || modifiers.mousecapture ? 'Capture' : ''
 
             addEvt(ctx, 'main', [
               [el, 'mousedown', 'mouseStart', `passive${capture}`]
             ])
           }
 
-          if (client.has.touch === true) {
+          if (client.has.touch) {
             addEvt(ctx, 'main', [
               [
                 el,
                 'touchstart',
                 'touchStart',
-                `passive${modifiers.capture === true ? 'Capture' : ''}`
+                `passive${modifiers.capture ? 'Capture' : ''}`
               ],
               [el, 'touchmove', 'noop', 'notPassiveCapture'] // cannot be passive (ex: iOS scroll)
             ])
@@ -458,7 +442,7 @@ export default createDirective(
             cleanEvt(ctx, 'main')
             cleanEvt(ctx, 'temp')
 
-            if (client.is.firefox === true) preventDraggable(el, false)
+            if (client.is.firefox) preventDraggable(el, false)
             ctx.styleCleanup?.()
 
             delete el.__qtouchpan

package/src/directives/touch-repeat/TouchRepeat.js

@@ -4,10 +4,10 @@ import { createDirective } from '../../utils/private.create/create.js'
 import {
   addEvt,
   cleanEvt,
-  position,
   leftClick,
-  stopAndPrevent,
-  noop
+  noop,
+  position,
+  stopAndPrevent
 } from '../../utils/event/event.js'
 import { clearSelection } from '../../utils/private.selection/selection.js'
 import { isKeyCode } from '../../utils/private.keyboard/key-composition.js'
@@ -28,10 +28,13 @@ const keyCodes = {
 
 function shouldEnd(evt, origin) {
   const { top, left } = position(evt)
-
   return Math.abs(left - origin.left) >= 7 || Math.abs(top - origin.top) >= 7
 }
 
+function removeBodyNonSelectable() {
+  document.body.classList.remove('non-selectable')
+}
+
 export default createDirective(
   __QUASAR_SSR_SERVER__
     ? { name: 'touch-repeat', getSSRProps }
@@ -40,10 +43,11 @@ export default createDirective(
 
         beforeMount(el, { modifiers, value, arg }) {
           const keyboard = Object.keys(modifiers).reduce((acc, key) => {
-            if (keyRegex.test(key) === true) {
-              const keyCode = isNaN(parseInt(key, 10))
+            if (keyRegex.test(key)) {
+              const parsedKey = Number.parseInt(key, 10)
+              const keyCode = Number.isNaN(parsedKey)
                 ? keyCodes[key.toLowerCase()]
-                : parseInt(key, 10)
+                : parsedKey
 
               if (keyCode >= 0) acc.push(keyCode)
             }
@@ -51,17 +55,13 @@ export default createDirective(
           }, [])
 
           // early return, we don't need to do anything
-          if (
-            modifiers.mouse !== true &&
-            client.has.touch !== true &&
-            keyboard.length === 0
-          ) {
+          if (!modifiers.mouse && !client.has.touch && keyboard.length === 0) {
             return
           }
 
           const durations =
             typeof arg === 'string' && arg.length !== 0
-              ? arg.split(':').map(val => parseInt(val, 10))
+              ? arg.split(':').map(val => Number.parseInt(val, 10))
               : [0, 600, 300]
 
           const durationsLast = durations.length - 1
@@ -76,7 +76,7 @@ export default createDirective(
               if (
                 ctx.event === void 0 &&
                 typeof ctx.handler === 'function' &&
-                leftClick(evt) === true
+                leftClick(evt)
               ) {
                 addEvt(ctx, 'temp', [
                   [document, 'mousemove', 'move', 'passiveCapture'],
@@ -89,7 +89,7 @@ export default createDirective(
             keyboardStart(evt) {
               if (
                 typeof ctx.handler === 'function' &&
-                isKeyCode(evt, keyboard) === true
+                isKeyCode(evt, keyboard)
               ) {
                 if (durations[0] === 0 || ctx.event !== void 0) {
                   stopAndPrevent(evt)
@@ -118,35 +118,28 @@ export default createDirective(
             },
 
             start(evt, mouseEvent, keyboardEvent) {
-              if (keyboardEvent !== true) {
-                ctx.origin = position(evt)
-              }
+              if (!keyboardEvent) ctx.origin = position(evt)
 
               function styleCleanup(withDelay) {
                 ctx.styleCleanup = void 0
-
                 document.documentElement.style.cursor = ''
 
-                const remove = () => {
-                  document.body.classList.remove('non-selectable')
-                }
-
                 if (withDelay === true) {
                   clearSelection()
-                  setTimeout(remove, 10)
+                  setTimeout(removeBodyNonSelectable, 10)
                 } else {
-                  remove()
+                  removeBodyNonSelectable()
                 }
               }
 
-              if (client.is.mobile === true) {
+              if (client.is.mobile) {
                 document.body.classList.add('non-selectable')
                 clearSelection()
                 ctx.styleCleanup = styleCleanup
               }
 
               ctx.event = {
-                touch: mouseEvent !== true && keyboardEvent !== true,
+                touch: !mouseEvent && !keyboardEvent,
                 mouse: mouseEvent === true,
                 keyboard: keyboardEvent === true,
                 startTime: Date.now(),
@@ -161,13 +154,13 @@ export default createDirective(
                 if (ctx.event.repeatCount === 0) {
                   ctx.event.evt = evt
 
-                  if (keyboardEvent === true) {
+                  if (keyboardEvent) {
                     ctx.event.keyCode = evt.keyCode
                   } else {
                     ctx.event.position = position(evt)
                   }
 
-                  if (client.is.mobile !== true) {
+                  if (!client.is.mobile) {
                     document.documentElement.style.cursor = 'pointer'
                     document.body.classList.add('non-selectable')
                     clearSelection()
@@ -199,7 +192,7 @@ export default createDirective(
               if (
                 ctx.event !== void 0 &&
                 ctx.timer !== void 0 &&
-                shouldEnd(evt, ctx.origin) === true
+                shouldEnd(evt, ctx.origin)
               ) {
                 clearTimeout(ctx.timer)
                 ctx.timer = void 0
@@ -227,25 +220,23 @@ export default createDirective(
 
           el.__qtouchrepeat = ctx
 
-          if (modifiers.mouse === true) {
+          if (modifiers.mouse) {
             // account for UMD too where modifiers will be lowercased to work
             const capture =
-              modifiers.mouseCapture === true || modifiers.mousecapture === true
-                ? 'Capture'
-                : ''
+              modifiers.mouseCapture || modifiers.mousecapture ? 'Capture' : ''
 
             addEvt(ctx, 'main', [
               [el, 'mousedown', 'mouseStart', `passive${capture}`]
             ])
           }
 
-          if (client.has.touch === true) {
+          if (client.has.touch) {
             addEvt(ctx, 'main', [
               [
                 el,
                 'touchstart',
                 'touchStart',
-                `passive${modifiers.capture === true ? 'Capture' : ''}`
+                `passive${modifiers.capture ? 'Capture' : ''}`
               ],
               [el, 'touchend', 'noop', 'passiveCapture']
             ])
@@ -254,9 +245,7 @@ export default createDirective(
           if (keyboard.length !== 0) {
             // account for UMD too where modifiers will be lowercased to work
             const capture =
-              modifiers.keyCapture === true || modifiers.keycapture === true
-                ? 'Capture'
-                : ''
+              modifiers.keyCapture || modifiers.keycapture ? 'Capture' : ''
 
             addEvt(ctx, 'main', [
               [el, 'keydown', 'keyboardStart', `notPassive${capture}`]