quasar 2.18.7 → 2.19.1

package/src/components/tabs/QTabs.js

@@ -1,4 +1,14 @@
-import { h, ref, computed, watch, onBeforeUnmount, onActivated, onDeactivated, getCurrentInstance, provide } from 'vue'
+import {
+  h,
+  ref,
+  computed,
+  watch,
+  onBeforeUnmount,
+  onActivated,
+  onDeactivated,
+  getCurrentInstance,
+  provide
+} from 'vue'
 
 import QIcon from '../icon/QIcon.js'
 import QResizeObserver from '../resize-observer/QResizeObserver.js'
@@ -11,21 +21,19 @@ import { hSlot } from '../../utils/private.render/render.js'
 import { tabsKey } from '../../utils/private.symbols/symbols.js'
 import { rtlHasScrollBug } from '../../utils/private.rtl/rtl.js'
 
-function getIndicatorClass (color, top, vertical) {
-  const pos = vertical === true
-    ? [ 'left', 'right' ]
-    : [ 'top', 'bottom' ]
+function getIndicatorClass(color, top, vertical) {
+  const pos = vertical === true ? ['left', 'right'] : ['top', 'bottom']
 
-  return `absolute-${ top === true ? pos[ 0 ] : pos[ 1 ] }${ color ? ` text-${ color }` : '' }`
+  return `absolute-${top === true ? pos[0] : pos[1]}${color ? ` text-${color}` : ''}`
 }
 
-const alignValues = [ 'left', 'center', 'right', 'justify' ]
+const alignValues = ['left', 'center', 'right', 'justify']
 
 export default createComponent({
   name: 'QTabs',
 
   props: {
-    modelValue: [ Number, String ],
+    modelValue: [Number, String],
 
     align: {
       type: String,
@@ -33,7 +41,7 @@ export default createComponent({
       validator: v => alignValues.includes(v)
     },
     breakpoint: {
-      type: [ String, Number ],
+      type: [String, Number],
       default: 600
     },
 
@@ -61,10 +69,10 @@ export default createComponent({
 
     contentClass: String,
 
-    'onUpdate:modelValue': [ Function, Array ]
+    'onUpdate:modelValue': [Function, Array]
   },
 
-  setup (props, { slots, emit }) {
+  setup(props, { slots, emit }) {
     const { proxy } = getCurrentInstance()
     const { $q } = proxy
 
@@ -72,8 +80,14 @@ export default createComponent({
     const { registerTick: registerUpdateArrowsTick } = useTick()
     const { registerTick: registerAnimateTick } = useTick()
 
-    const { registerTimeout: registerFocusTimeout, removeTimeout: removeFocusTimeout } = useTimeout()
-    const { registerTimeout: registerScrollToTabTimeout, removeTimeout: removeScrollToTabTimeout } = useTimeout()
+    const {
+      registerTimeout: registerFocusTimeout,
+      removeTimeout: removeFocusTimeout
+    } = useTimeout()
+    const {
+      registerTimeout: registerScrollToTabTimeout,
+      removeTimeout: removeScrollToTabTimeout
+    } = useTimeout()
 
     const rootRef = ref(null)
     const contentRef = ref(null)
@@ -88,7 +102,9 @@ export default createComponent({
     const tabDataListLen = ref(0)
     const hasFocus = ref(false)
 
-    let animateTimer = null, scrollTimer = null, unwatchRoute
+    let animateTimer = null,
+      scrollTimer = null,
+      unwatchRoute
 
     const tabProps = computed(() => ({
       activeClass: props.activeClass,
@@ -109,7 +125,7 @@ export default createComponent({
       const val = currentModel.value
 
       for (let i = 0; i < len; i++) {
-        if (tabDataList[ i ].name.value === val) {
+        if (tabDataList[i].name.value === val) {
           return true
         }
       }
@@ -118,85 +134,99 @@ export default createComponent({
     })
 
     const alignClass = computed(() => {
-      const align = scrollable.value === true
-        ? 'left'
-        : (justify.value === true ? 'justify' : props.align)
-
-      return `q-tabs__content--align-${ align }`
+      const align =
+        scrollable.value === true
+          ? 'left'
+          : justify.value === true
+            ? 'justify'
+            : props.align
+
+      return `q-tabs__content--align-${align}`
     })
 
-    const classes = computed(() =>
-      'q-tabs row no-wrap items-center'
-      + ` q-tabs--${ scrollable.value === true ? '' : 'not-' }scrollable`
-      + ` q-tabs--${ props.vertical === true ? 'vertical' : 'horizontal' }`
-      + ` q-tabs__arrows--${ props.outsideArrows === true ? 'outside' : 'inside' }`
-      + ` q-tabs--mobile-with${ props.mobileArrows === true ? '' : 'out' }-arrows`
-      + (props.dense === true ? ' q-tabs--dense' : '')
-      + (props.shrink === true ? ' col-shrink' : '')
-      + (props.stretch === true ? ' self-stretch' : '')
+    const classes = computed(
+      () =>
+        'q-tabs row no-wrap items-center' +
+        ` q-tabs--${scrollable.value === true ? '' : 'not-'}scrollable` +
+        ` q-tabs--${props.vertical === true ? 'vertical' : 'horizontal'}` +
+        ` q-tabs__arrows--${props.outsideArrows === true ? 'outside' : 'inside'}` +
+        ` q-tabs--mobile-with${props.mobileArrows === true ? '' : 'out'}-arrows` +
+        (props.dense === true ? ' q-tabs--dense' : '') +
+        (props.shrink === true ? ' col-shrink' : '') +
+        (props.stretch === true ? ' self-stretch' : '')
     )
 
-    const innerClass = computed(() =>
-      'q-tabs__content scroll--mobile row no-wrap items-center self-stretch hide-scrollbar relative-position '
-      + alignClass.value
-      + (props.contentClass !== void 0 ? ` ${ props.contentClass }` : '')
+    const innerClass = computed(
+      () =>
+        'q-tabs__content scroll--mobile row no-wrap items-center self-stretch hide-scrollbar relative-position ' +
+        alignClass.value +
+        (props.contentClass !== void 0 ? ` ${props.contentClass}` : '')
     )
 
-    const domProps = computed(() => (
+    const domProps = computed(() =>
       props.vertical === true
-        ? { container: 'height', content: 'offsetHeight', scroll: 'scrollHeight' }
+        ? {
+            container: 'height',
+            content: 'offsetHeight',
+            scroll: 'scrollHeight'
+          }
         : { container: 'width', content: 'offsetWidth', scroll: 'scrollWidth' }
-    ))
+    )
 
-    const isRTL = computed(() => props.vertical !== true && $q.lang.rtl === true)
-    const rtlPosCorrection = computed(() => rtlHasScrollBug === false && isRTL.value === true)
+    const isRTL = computed(
+      () => props.vertical !== true && $q.lang.rtl === true
+    )
+    const rtlPosCorrection = computed(
+      () => rtlHasScrollBug === false && isRTL.value === true
+    )
 
     watch(isRTL, updateArrows)
 
-    watch(() => props.modelValue, name => {
-      updateModel({ name, setCurrent: true, skipEmit: true })
-    })
+    watch(
+      () => props.modelValue,
+      name => {
+        updateModel({ name, setCurrent: true, skipEmit: true })
+      }
+    )
 
     watch(() => props.outsideArrows, recalculateScroll)
 
-    function updateModel ({ name, setCurrent, skipEmit }) {
+    function updateModel({ name, setCurrent, skipEmit }) {
       if (currentModel.value === name) return
 
-      if (skipEmit !== true && props[ 'onUpdate:modelValue' ] !== void 0) {
+      if (skipEmit !== true && props['onUpdate:modelValue'] !== void 0) {
         emit('update:modelValue', name)
       }
 
-      if (
-        setCurrent === true
-        || props[ 'onUpdate:modelValue' ] === void 0
-      ) {
+      if (setCurrent === true || props['onUpdate:modelValue'] === void 0) {
         animate(currentModel.value, name)
         currentModel.value = name
       }
     }
 
-    function recalculateScroll () {
+    function recalculateScroll() {
       registerScrollTick(() => {
-        rootRef.value && updateContainer({
-          width: rootRef.value.offsetWidth,
-          height: rootRef.value.offsetHeight
-        })
+        if (rootRef.value) {
+          updateContainer({
+            width: rootRef.value.offsetWidth,
+            height: rootRef.value.offsetHeight
+          })
+        }
       })
     }
 
-    function updateContainer (domSize) {
+    function updateContainer(domSize) {
       // it can be called faster than component being initialized
       // so we need to protect against that case
       // (one example of such case is the docs release notes page)
       if (domProps.value === void 0 || contentRef.value === null) return
 
-      const
-        size = domSize[ domProps.value.container ],
+      const size = domSize[domProps.value.container],
         scrollSize = Math.min(
-          contentRef.value[ domProps.value.scroll ],
+          contentRef.value[domProps.value.scroll],
           Array.prototype.reduce.call(
             contentRef.value.children,
-            (acc, el) => acc + (el[ domProps.value.content ] || 0),
+            (acc, el) => acc + (el[domProps.value.content] || 0),
             0
           )
         ),
@@ -205,29 +235,28 @@ export default createComponent({
       scrollable.value = scroll
 
       // Arrows need to be updated even if the scroll status was already true
-      scroll === true && registerUpdateArrowsTick(updateArrows)
+      if (scroll === true) registerUpdateArrowsTick(updateArrows)
 
       justify.value = size < parseInt(props.breakpoint, 10)
     }
 
-    function animate (oldName, newName) {
-      const
-        oldTab = oldName !== void 0 && oldName !== null && oldName !== ''
-          ? tabDataList.find(tab => tab.name.value === oldName)
-          : null,
-        newTab = newName !== void 0 && newName !== null && newName !== ''
-          ? tabDataList.find(tab => tab.name.value === newName)
-          : null
+    function animate(oldName, newName) {
+      const oldTab =
+          oldName !== void 0 && oldName !== null && oldName !== ''
+            ? tabDataList.find(tab => tab.name.value === oldName)
+            : null,
+        newTab =
+          newName !== void 0 && newName !== null && newName !== ''
+            ? tabDataList.find(tab => tab.name.value === newName)
+            : null
 
       if (hadActivated === true) {
         // After the component has been re-activated
         // we should not animate the transition.
         // Consider it as if the component has just been mounted.
         hadActivated = false
-      }
-      else if (oldTab && newTab) {
-        const
-          oldEl = oldTab.tabIndicatorRef.value,
+      } else if (oldTab && newTab) {
+        const oldEl = oldTab.tabIndicatorRef.value,
           newEl = newTab.tabIndicatorRef.value
 
         if (animateTimer !== null) {
@@ -240,13 +269,13 @@ export default createComponent({
         newEl.style.transition = 'none'
         newEl.style.transform = 'none'
 
-        const
-          oldPos = oldEl.getBoundingClientRect(),
+        const oldPos = oldEl.getBoundingClientRect(),
           newPos = newEl.getBoundingClientRect()
 
-        newEl.style.transform = props.vertical === true
-          ? `translate3d(0,${ oldPos.top - newPos.top }px,0) scale3d(1,${ newPos.height ? oldPos.height / newPos.height : 1 },1)`
-          : `translate3d(${ oldPos.left - newPos.left }px,0,0) scale3d(${ newPos.width ? oldPos.width / newPos.width : 1 },1,1)`
+        newEl.style.transform =
+          props.vertical === true
+            ? `translate3d(0,${oldPos.top - newPos.top}px,0) scale3d(1,${newPos.height ? oldPos.height / newPos.height : 1},1)`
+            : `translate3d(${oldPos.left - newPos.left}px,0,0) scale3d(${newPos.width ? oldPos.width / newPos.width : 1},1,1)`
 
         // allow scope updates to kick in (QRouteTab needs more time)
         registerAnimateTick(() => {
@@ -263,48 +292,56 @@ export default createComponent({
       }
     }
 
-    function scrollToTabEl (el) {
-      const
-        { left, width, top, height } = contentRef.value.getBoundingClientRect(),
+    function scrollToTabEl(el) {
+      const { left, width, top, height } =
+          contentRef.value.getBoundingClientRect(),
         newPos = el.getBoundingClientRect()
 
-      let offset = props.vertical === true ? newPos.top - top : newPos.left - left
+      let offset =
+        props.vertical === true ? newPos.top - top : newPos.left - left
 
       if (offset < 0) {
-        contentRef.value[ props.vertical === true ? 'scrollTop' : 'scrollLeft' ] += Math.floor(offset)
+        contentRef.value[
+          props.vertical === true ? 'scrollTop' : 'scrollLeft'
+        ] += Math.floor(offset)
         updateArrows()
         return
       }
 
-      offset += props.vertical === true ? newPos.height - height : newPos.width - width
+      offset +=
+        props.vertical === true ? newPos.height - height : newPos.width - width
       if (offset > 0) {
-        contentRef.value[ props.vertical === true ? 'scrollTop' : 'scrollLeft' ] += Math.ceil(offset)
+        contentRef.value[
+          props.vertical === true ? 'scrollTop' : 'scrollLeft'
+        ] += Math.ceil(offset)
         updateArrows()
       }
     }
 
-    function updateArrows () {
+    function updateArrows() {
       const content = contentRef.value
       if (content === null) return
 
-      const
-        rect = content.getBoundingClientRect(),
-        pos = props.vertical === true ? content.scrollTop : Math.abs(content.scrollLeft)
+      const rect = content.getBoundingClientRect(),
+        pos =
+          props.vertical === true
+            ? content.scrollTop
+            : Math.abs(content.scrollLeft)
 
       if (isRTL.value === true) {
         leftArrow.value = Math.ceil(pos + rect.width) < content.scrollWidth - 1
         rightArrow.value = pos > 0
-      }
-      else {
+      } else {
         leftArrow.value = pos > 0
-        rightArrow.value = props.vertical === true
-          ? Math.ceil(pos + rect.height) < content.scrollHeight
-          : Math.ceil(pos + rect.width) < content.scrollWidth
+        rightArrow.value =
+          props.vertical === true
+            ? Math.ceil(pos + rect.height) < content.scrollHeight
+            : Math.ceil(pos + rect.width) < content.scrollWidth
       }
     }
 
-    function animScrollTo (value) {
-      scrollTimer !== null && clearInterval(scrollTimer)
+    function animScrollTo(value) {
+      if (scrollTimer !== null) clearInterval(scrollTimer)
       scrollTimer = setInterval(() => {
         if (scrollTowards(value) === true) {
           stopAnimScroll()
@@ -312,53 +349,65 @@ export default createComponent({
       }, 5)
     }
 
-    function scrollToStart () {
-      animScrollTo(rtlPosCorrection.value === true ? Number.MAX_SAFE_INTEGER : 0)
+    function scrollToStart() {
+      animScrollTo(
+        rtlPosCorrection.value === true ? Number.MAX_SAFE_INTEGER : 0
+      )
     }
 
-    function scrollToEnd () {
-      animScrollTo(rtlPosCorrection.value === true ? 0 : Number.MAX_SAFE_INTEGER)
+    function scrollToEnd() {
+      animScrollTo(
+        rtlPosCorrection.value === true ? 0 : Number.MAX_SAFE_INTEGER
+      )
     }
 
-    function stopAnimScroll () {
+    function stopAnimScroll() {
       if (scrollTimer !== null) {
         clearInterval(scrollTimer)
         scrollTimer = null
       }
     }
 
-    function onKbdNavigate (keyCode, fromEl) {
+    function onKbdNavigate(keyCode, fromEl) {
       const tabs = Array.prototype.filter.call(
         contentRef.value.children,
-        el => el === fromEl || (el.matches && el.matches('.q-tab.q-focusable') === true)
+        el =>
+          el === fromEl ||
+          (el.matches && el.matches('.q-tab.q-focusable') === true)
       )
 
       const len = tabs.length
       if (len === 0) return
 
-      if (keyCode === 36) { // Home
-        scrollToTabEl(tabs[ 0 ])
-        tabs[ 0 ].focus()
+      if (keyCode === 36) {
+        // Home
+        scrollToTabEl(tabs[0])
+        tabs[0].focus()
         return true
       }
-      if (keyCode === 35) { // End
-        scrollToTabEl(tabs[ len - 1 ])
-        tabs[ len - 1 ].focus()
+      if (keyCode === 35) {
+        // End
+        scrollToTabEl(tabs[len - 1])
+        tabs[len - 1].focus()
         return true
       }
 
-      const dirPrev = keyCode === (props.vertical === true ? 38 /* ArrowUp */ : 37 /* ArrowLeft */)
-      const dirNext = keyCode === (props.vertical === true ? 40 /* ArrowDown */ : 39 /* ArrowRight */)
+      const dirPrev =
+        keyCode ===
+        (props.vertical === true ? 38 /* ArrowUp */ : 37) /* ArrowLeft */
+      const dirNext =
+        keyCode ===
+        (props.vertical === true ? 40 /* ArrowDown */ : 39) /* ArrowRight */
 
-      const dir = dirPrev === true ? -1 : (dirNext === true ? 1 : void 0)
+      const dir = dirPrev === true ? -1 : dirNext === true ? 1 : void 0
 
       if (dir !== void 0) {
         const rtlDir = isRTL.value === true ? -1 : 1
         const index = tabs.indexOf(fromEl) + dir * rtlDir
 
         if (index >= 0 && index < len) {
-          scrollToTabEl(tabs[ index ])
-          tabs[ index ].focus({ preventScroll: true })
+          scrollToTabEl(tabs[index])
+          tabs[index].focus({ preventScroll: true })
         }
 
         return true
@@ -368,23 +417,34 @@ export default createComponent({
     // let's speed up execution of time-sensitive scrollTowards()
     // with a computed variable by directly applying the minimal
     // number of instructions on get/set functions
-    const posFn = computed(() => (
+    const posFn = computed(() =>
       rtlPosCorrection.value === true
-        ? { get: content => Math.abs(content.scrollLeft), set: (content, pos) => { content.scrollLeft = -pos } }
-        : (
-            props.vertical === true
-              ? { get: content => content.scrollTop, set: (content, pos) => { content.scrollTop = pos } }
-              : { get: content => content.scrollLeft, set: (content, pos) => { content.scrollLeft = pos } }
-          )
-    ))
+        ? {
+            get: content => Math.abs(content.scrollLeft),
+            set: (content, pos) => {
+              content.scrollLeft = -pos
+            }
+          }
+        : props.vertical === true
+          ? {
+              get: content => content.scrollTop,
+              set: (content, pos) => {
+                content.scrollTop = pos
+              }
+            }
+          : {
+              get: content => content.scrollLeft,
+              set: (content, pos) => {
+                content.scrollLeft = pos
+              }
+            }
+    )
 
-    function scrollTowards (value) {
-      const
-        content = contentRef.value,
+    function scrollTowards(value) {
+      const content = contentRef.value,
         { get, set } = posFn.value
 
-      let
-        done = false,
+      let done = false,
         pos = get(content)
 
       const direction = value < pos ? -1 : 1
@@ -394,10 +454,9 @@ export default createComponent({
       if (pos < 0) {
         done = true
         pos = 0
-      }
-      else if (
-        (direction === -1 && pos <= value)
-        || (direction === 1 && pos >= value)
+      } else if (
+        (direction === -1 && pos <= value) ||
+        (direction === 1 && pos >= value)
       ) {
         done = true
         pos = value
@@ -409,9 +468,9 @@ export default createComponent({
       return done
     }
 
-    function hasQueryIncluded (targetQuery, matchingQuery) {
+    function hasQueryIncluded(targetQuery, matchingQuery) {
       for (const key in targetQuery) {
-        if (targetQuery[ key ] !== matchingQuery[ key ]) {
+        if (targetQuery[key] !== matchingQuery[key]) {
           return false
         }
       }
@@ -421,10 +480,13 @@ export default createComponent({
 
     // 1. Do not use directly; use verifyRouteModel() instead
     // 2. Should set hadActivated to false upon exit
-    function updateActiveRoute () {
-      let name = null, bestScore = { matchedLen: 0, queryDiff: 9999, hrefLen: 0 }
+    function updateActiveRoute() {
+      let name = null,
+        bestScore = { matchedLen: 0, queryDiff: 9999, hrefLen: 0 }
 
-      const list = tabDataList.filter(tab => tab.routeData?.hasRouterLink.value === true)
+      const list = tabDataList.filter(
+        tab => tab.routeData?.hasRouterLink.value === true
+      )
       const { hash: currentHash, query: currentQuery } = proxy.$route
       const currentQueryLen = Object.keys(currentQuery).length
 
@@ -434,7 +496,10 @@ export default createComponent({
       for (const tab of list) {
         const exact = tab.routeData.exact.value === true
 
-        if (tab.routeData[ exact === true ? 'linkIsExactActive' : 'linkIsActive' ].value !== true) {
+        if (
+          tab.routeData[exact === true ? 'linkIsExactActive' : 'linkIsActive']
+            .value !== true
+        ) {
           // it cannot match anything as it's not active nor exact-active
           continue
         }
@@ -449,8 +514,8 @@ export default createComponent({
           }
 
           if (
-            queryLen !== currentQueryLen
-            || hasQueryIncluded(currentQuery, query) === false
+            queryLen !== currentQueryLen ||
+            hasQueryIncluded(currentQuery, query) === false
           ) {
             // it's set to exact but it doesn't matches the query
             continue
@@ -466,10 +531,7 @@ export default createComponent({
           continue
         }
 
-        if (
-          queryLen !== 0
-          && hasQueryIncluded(query, currentQuery) === false
-        ) {
+        if (queryLen !== 0 && hasQueryIncluded(query, currentQuery) === false) {
           // it has query and it doesn't includes the current one
           continue
         }
@@ -485,8 +547,7 @@ export default createComponent({
           name = tab.name.value
           bestScore = newScore
           continue
-        }
-        else if (newScore.matchedLen !== bestScore.matchedLen) {
+        } else if (newScore.matchedLen !== bestScore.matchedLen) {
           // it matches less routes than the current champion so we discard it
           continue
         }
@@ -495,8 +556,7 @@ export default createComponent({
           // query is closer to the current one so we set it as current champion
           name = tab.name.value
           bestScore = newScore
-        }
-        else if (newScore.queryDiff !== bestScore.queryDiff) {
+        } else if (newScore.queryDiff !== bestScore.queryDiff) {
           // it matches less routes than the current champion so we discard it
           continue
         }
@@ -509,8 +569,11 @@ export default createComponent({
       }
 
       if (
-        name === null
-        && tabDataList.some(tab => tab.routeData === void 0 && tab.name.value === currentModel.value) === true
+        name === null &&
+        tabDataList.some(
+          tab =>
+            tab.routeData === void 0 && tab.name.value === currentModel.value
+        ) === true
       ) {
         // we shouldn't interfere if non-route tab is active
         hadActivated = false
@@ -520,14 +583,14 @@ export default createComponent({
       updateModel({ name, setCurrent: true })
     }
 
-    function onFocusin (e) {
+    function onFocusin(e) {
       removeFocusTimeout()
 
       if (
-        hasFocus.value !== true
-        && rootRef.value !== null
-        && e.target
-        && typeof e.target.closest === 'function'
+        hasFocus.value !== true &&
+        rootRef.value !== null &&
+        e.target &&
+        typeof e.target.closest === 'function'
       ) {
         const tab = e.target.closest('.q-tab')
 
@@ -535,25 +598,26 @@ export default createComponent({
         // (it might be other elements focused, like additional QBtn)
         if (tab && rootRef.value.contains(tab) === true) {
           hasFocus.value = true
-          scrollable.value === true && scrollToTabEl(tab)
+          if (scrollable.value === true) scrollToTabEl(tab)
         }
       }
     }
 
-    function onFocusout () {
-      registerFocusTimeout(() => { hasFocus.value = false }, 30)
+    function onFocusout() {
+      registerFocusTimeout(() => {
+        hasFocus.value = false
+      }, 30)
     }
 
-    function verifyRouteModel () {
+    function verifyRouteModel() {
       if ($tabs.avoidRouteWatcher === false) {
         registerScrollToTabTimeout(updateActiveRoute)
-      }
-      else {
+      } else {
         removeScrollToTabTimeout()
       }
     }
 
-    function watchRoute () {
+    function watchRoute() {
       if (unwatchRoute === void 0) {
         const unwatch = watch(() => proxy.$route.fullPath, verifyRouteModel)
         unwatchRoute = () => {
@@ -563,7 +627,7 @@ export default createComponent({
       }
     }
 
-    function registerTab (tabData) {
+    function registerTab(tabData) {
       tabDataList.push(tabData)
       tabDataListLen.value++
 
@@ -575,11 +639,12 @@ export default createComponent({
         registerScrollToTabTimeout(() => {
           if (scrollable.value === true) {
             const value = currentModel.value
-            const newTab = value !== void 0 && value !== null && value !== ''
-              ? tabDataList.find(tab => tab.name.value === value)
-              : null
+            const newTab =
+              value !== void 0 && value !== null && value !== ''
+                ? tabDataList.find(tab => tab.name.value === value)
+                : null
 
-            newTab && scrollToTabEl(newTab.rootRef.value)
+            if (newTab) scrollToTabEl(newTab.rootRef.value)
           }
         })
       }
@@ -594,7 +659,7 @@ export default createComponent({
       }
     }
 
-    function unregisterTab (tabData) {
+    function unregisterTab(tabData) {
       tabDataList.splice(tabDataList.indexOf(tabData), 1)
       tabDataListLen.value--
 
@@ -629,8 +694,8 @@ export default createComponent({
 
     provide(tabsKey, $tabs)
 
-    function cleanup () {
-      animateTimer !== null && clearTimeout(animateTimer)
+    function cleanup() {
+      if (animateTimer !== null) clearTimeout(animateTimer)
       stopAnimScroll()
       unwatchRoute?.()
     }
@@ -654,44 +719,57 @@ export default createComponent({
       recalculateScroll()
     })
 
-    return () => {
-      return h('div', {
-        ref: rootRef,
-        class: classes.value,
-        role: 'tablist',
-        onFocusin,
-        onFocusout
-      }, [
-        h(QResizeObserver, { onResize: updateContainer }),
-
-        h('div', {
-          ref: contentRef,
-          class: innerClass.value,
-          onScroll: updateArrows
-        }, hSlot(slots.default)),
-
-        h(QIcon, {
-          class: 'q-tabs__arrow q-tabs__arrow--left absolute q-tab__icon'
-            + (leftArrow.value === true ? '' : ' q-tabs__arrow--faded'),
-          name: props.leftIcon || $q.iconSet.tabs[ props.vertical === true ? 'up' : 'left' ],
-          onMousedownPassive: scrollToStart,
-          onTouchstartPassive: scrollToStart,
-          onMouseupPassive: stopAnimScroll,
-          onMouseleavePassive: stopAnimScroll,
-          onTouchendPassive: stopAnimScroll
-        }),
-
-        h(QIcon, {
-          class: 'q-tabs__arrow q-tabs__arrow--right absolute q-tab__icon'
-            + (rightArrow.value === true ? '' : ' q-tabs__arrow--faded'),
-          name: props.rightIcon || $q.iconSet.tabs[ props.vertical === true ? 'down' : 'right' ],
-          onMousedownPassive: scrollToEnd,
-          onTouchstartPassive: scrollToEnd,
-          onMouseupPassive: stopAnimScroll,
-          onMouseleavePassive: stopAnimScroll,
-          onTouchendPassive: stopAnimScroll
-        })
-      ])
-    }
+    return () =>
+      h(
+        'div',
+        {
+          ref: rootRef,
+          class: classes.value,
+          role: 'tablist',
+          onFocusin,
+          onFocusout
+        },
+        [
+          h(QResizeObserver, { onResize: updateContainer }),
+
+          h(
+            'div',
+            {
+              ref: contentRef,
+              class: innerClass.value,
+              onScroll: updateArrows
+            },
+            hSlot(slots.default)
+          ),
+
+          h(QIcon, {
+            class:
+              'q-tabs__arrow q-tabs__arrow--left absolute q-tab__icon' +
+              (leftArrow.value === true ? '' : ' q-tabs__arrow--faded'),
+            name:
+              props.leftIcon ||
+              $q.iconSet.tabs[props.vertical === true ? 'up' : 'left'],
+            onMousedownPassive: scrollToStart,
+            onTouchstartPassive: scrollToStart,
+            onMouseupPassive: stopAnimScroll,
+            onMouseleavePassive: stopAnimScroll,
+            onTouchendPassive: stopAnimScroll
+          }),
+
+          h(QIcon, {
+            class:
+              'q-tabs__arrow q-tabs__arrow--right absolute q-tab__icon' +
+              (rightArrow.value === true ? '' : ' q-tabs__arrow--faded'),
+            name:
+              props.rightIcon ||
+              $q.iconSet.tabs[props.vertical === true ? 'down' : 'right'],
+            onMousedownPassive: scrollToEnd,
+            onTouchstartPassive: scrollToEnd,
+            onMouseupPassive: stopAnimScroll,
+            onMouseleavePassive: stopAnimScroll,
+            onTouchendPassive: stopAnimScroll
+          })
+        ]
+      )
   }
 })