From 715ed63724dd2f30b87ac212c1ad80ee1fd263a1 Mon Sep 17 00:00:00 2001
From: Dan Popescu <pdan.popescu@gmail.com>
Date: Mon, 14 Feb 2022 19:18:21 +0200
Subject: [PATCH] feat(QSplitter): add keyboard navigation #12466

#12466
---
 ui/dev/src/pages/components/splitter.vue  |  29 +++--
 ui/src/components/splitter/QSplitter.js   | 123 ++++++++++++++++++----
 ui/src/components/splitter/QSplitter.json |   5 +
 3 files changed, 128 insertions(+), 29 deletions(-)

diff --git a/ui/dev/src/pages/components/splitter.vue b/ui/dev/src/pages/components/splitter.vue
index da562568c35f..37f0a740751b 100644
--- a/ui/dev/src/pages/components/splitter.vue
+++ b/ui/dev/src/pages/components/splitter.vue
@@ -40,7 +40,9 @@
       style="height: 700px; border: 1px solid black"
     >
       <template v-slot:before>
-        <div class="q-layout-padding">
+        <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+          <div class="q-focus-helper" />
+
           <div class="text-h1 q-mb-md">
             Before
           </div>
@@ -60,7 +62,9 @@
       />
 
       <template v-slot:after>
-        <div class="q-layout-padding">
+        <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+          <div class="q-focus-helper" />
+
           <div class="text-h1 q-mb-md">
             After
           </div>
@@ -83,7 +87,9 @@
       separator-class="bg-deep-orange"
     >
       <template v-slot:before>
-        <div class="q-layout-padding">
+        <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+          <div class="q-focus-helper" />
+
           <div class="text-h1 q-mb-md">
             Before
           </div>
@@ -109,9 +115,12 @@
           separator-class="bg-deep-orange"
           class="bg-white rounded-borders"
           style="width: 50vw; height: 30vh"
+          @keydown.stop
         >
           <template v-slot:before>
-            <div class="q-layout-padding">
+            <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+              <div class="q-focus-helper" />
+
               <div v-for="n in 20" :key="n" class="q-my-md">
                 {{ n }}. Lorem ipsum dolor sit.
               </div>
@@ -119,7 +128,9 @@
           </template>
 
           <template v-slot:after>
-            <div class="q-layout-padding">
+            <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+              <div class="q-focus-helper" />
+
               <div v-for="n in 20" :key="n" class="q-my-md">
                 {{ n }}. Lorem ipsum dolor sit.
               </div>
@@ -137,7 +148,9 @@
           separator-class="bg-deep-orange"
         >
           <template v-slot:before>
-            <div class="q-layout-padding">
+            <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+              <div class="q-focus-helper" />
+
               <div class="text-h1 q-mb-md">
                 After - Before
               </div>
@@ -158,7 +171,9 @@
           />
 
           <template v-slot:after>
-            <div class="q-layout-padding">
+            <div class="q-layout-padding q-focusable relative-position" tabindex="0">
+              <div class="q-focus-helper" />
+
               <div class="text-h1 q-mb-md">
                 After - After
               </div>
diff --git a/ui/src/components/splitter/QSplitter.js b/ui/src/components/splitter/QSplitter.js
index d5ab5edff25e..5ad3c7bac4a9 100644
--- a/ui/src/components/splitter/QSplitter.js
+++ b/ui/src/components/splitter/QSplitter.js
@@ -6,9 +6,16 @@ import DarkMixin from '../../mixins/dark.js'
 import ListenersMixin from '../../mixins/listeners.js'
 
 import { slot, mergeSlot } from '../../utils/slot.js'
-import { stop } from '../../utils/event.js'
+import { stop, stopAndPrevent } from '../../utils/event.js'
 import cache from '../../utils/cache.js'
 
+const keyDirections = {
+  37: 'left',
+  38: 'up',
+  39: 'right',
+  40: 'down'
+}
+
 export default Vue.extend({
   name: 'QSplitter',
 
@@ -44,6 +51,8 @@ export default Vue.extend({
     horizontal: Boolean,
     disable: Boolean,
 
+    tabindex: [String, Number],
+
     beforeClass: [Array, String, Object],
     afterClass: [Array, String, Object],
 
@@ -113,31 +122,63 @@ export default Vue.extend({
           }
         }]
       }
+    },
+
+    separatorAttrs () {
+      return this.disable === true
+        ? { tabindex: -1, 'aria-disabled': 'true' }
+        : { tabindex: this.tabindex || 0 }
+    },
+
+    separatorEvents () {
+      return this.disable === true
+        ? void 0
+        : { keydown: this.__panKeydown }
     }
   },
 
   methods: {
+    __panStart () {
+      const size = this.$el.getBoundingClientRect()[this.prop]
+
+      this.__dir = this.horizontal === true ? 'up' : 'left'
+      this.__maxValue = this.unit === '%' ? 100 : size
+      this.__value = Math.min(this.__maxValue, this.computedLimits[1], Math.max(this.computedLimits[0], this.value))
+      this.__multiplier = (this.reverse !== true ? 1 : -1) *
+        (this.horizontal === true ? 1 : (this.$q.lang.rtl === true ? -1 : 1)) *
+        (this.unit === '%' ? (size === 0 ? 0 : 100 / size) : 1)
+
+      this.$el.classList.add('q-splitter--active')
+    },
+
+    __panProgress (val) {
+      this.__normalized = Math.min(this.__maxValue, this.computedLimits[1], Math.max(this.computedLimits[0], val))
+
+      this.$refs[this.side].style[this.prop] = this.__getCSSValue(this.__normalized)
+
+      if (this.emitImmediately === true && this.value !== this.__normalized) {
+        this.$emit('input', this.__normalized)
+      }
+    },
+
+    __panEnd () {
+      this.__panCleanup !== void 0 && this.__panCleanup()
+
+      if (this.__normalized !== this.value) {
+        this.$emit('input', this.__normalized)
+      }
+
+      this.$el.classList.remove('q-splitter--active')
+    },
+
     __pan (evt) {
       if (evt.isFinal === true) {
-        if (this.__normalized !== this.value) {
-          this.$emit('input', this.__normalized)
-        }
-
-        this.$el.classList.remove('q-splitter--active')
+        this.__panEnd()
         return
       }
 
       if (evt.isFirst === true) {
-        const size = this.$el.getBoundingClientRect()[this.prop]
-
-        this.__dir = this.horizontal === true ? 'up' : 'left'
-        this.__maxValue = this.unit === '%' ? 100 : size
-        this.__value = Math.min(this.__maxValue, this.computedLimits[1], Math.max(this.computedLimits[0], this.value))
-        this.__multiplier = (this.reverse !== true ? 1 : -1) *
-          (this.horizontal === true ? 1 : (this.$q.lang.rtl === true ? -1 : 1)) *
-          (this.unit === '%' ? (size === 0 ? 0 : 100 / size) : 1)
-
-        this.$el.classList.add('q-splitter--active')
+        this.__panStart()
         return
       }
 
@@ -146,13 +187,47 @@ export default Vue.extend({
         (evt.direction === this.__dir ? -1 : 1) *
         evt.distance[this.horizontal === true ? 'y' : 'x']
 
-      this.__normalized = Math.min(this.__maxValue, this.computedLimits[1], Math.max(this.computedLimits[0], val))
+      this.__panProgress(val)
+    },
 
-      this.$refs[this.side].style[this.prop] = this.__getCSSValue(this.__normalized)
+    __panKeydown (evt) {
+      this.qListeners.keydown !== void 0 && this.$emit('keydown', evt)
 
-      if (this.emitImmediately === true && this.value !== this.__normalized) {
-        this.$emit('input', this.__normalized)
+      if (
+        this.disable === true ||
+        evt.defaultPrevented === true ||
+        (this.horizontal !== true && [ 37, 39 ].indexOf(evt.keyCode) === -1) ||
+        (this.horizontal === true && [ 38, 40 ].indexOf(evt.keyCode) === -1)
+      ) {
+        return
+      }
+
+      stopAndPrevent(evt)
+
+      if (this.__panCleanup === void 0) {
+        document.addEventListener('keyup', this.__panEnd)
+        document.addEventListener('focusout', this.__panEnd)
+
+        this.__panCleanup = () => {
+          this.__panCleanup = void 0
+          document.removeEventListener('keyup', this.__panEnd)
+          document.removeEventListener('focusout', this.__panEnd)
+
+          this.__panEnd()
+        }
+
+        this.__panStart()
+        this.__normalized = this.__value
       }
+
+      const direction = keyDirections[evt.keyCode]
+
+      const val = this.__normalized +
+        this.__multiplier *
+        (direction === this.__dir ? -1 : 1) *
+        (evt.shiftKey === true ? 1 : 10)
+
+      this.__panProgress(val)
     },
 
     __normalize (val, limits) {
@@ -169,8 +244,11 @@ export default Vue.extend({
     }
   },
 
+  beforeDestroy () {
+    this.__panCleanup !== void 0 && this.__panCleanup()
+  },
+
   render (h) {
-    const attrs = this.disable === true ? { 'aria-disabled': 'true' } : void 0
     const child = [
       h('div', {
         ref: 'before',
@@ -184,7 +262,8 @@ export default Vue.extend({
         staticClass: 'q-splitter__separator',
         style: this.separatorStyle,
         class: this.separatorClass,
-        attrs
+        attrs: this.separatorAttrs,
+        on: this.separatorEvents
       }, [
         h('div', {
           staticClass: 'absolute-full q-splitter__separator-area',
diff --git a/ui/src/components/splitter/QSplitter.json b/ui/src/components/splitter/QSplitter.json
index 58a9ac95f0ef..cc80aa08012e 100644
--- a/ui/src/components/splitter/QSplitter.json
+++ b/ui/src/components/splitter/QSplitter.json
@@ -51,6 +51,11 @@
       "category": "content|model"
     },
 
+    "tabindex": {
+      "extends": "tabindex",
+      "addedIn": "v1.18.6"
+    },
+
     "disable": {
       "extends": "disable"
     },