Lost focus in SQL query editor #54

src/views/Main/Editor/Tabs/Tab/SqlEditor/index.vue

@@ -1,6 +1,6 @@
 <template>
   <div class="codemirror-container">
-    <codemirror v-model="query" :options="cmOptions" @changes="onChange" />
+    <codemirror ref="cm" v-model="query" :options="cmOptions" @changes="onChange" />
   </div>
 </template>
 
@@ -28,7 +28,6 @@ export default {
         theme: 'neo',
         lineNumbers: true,
         line: true,
-        autofocus: true,
         autoRefresh: true,
         extraKeys: { 'Ctrl-Space': showHintOnDemand }
       }
@@ -40,7 +39,10 @@ export default {
     }
   },
   methods: {
-    onChange: debounce(showHint, 400)
+    onChange: debounce(showHint, 400),
+    focus () {
+      this.$refs.cm.codemirror.focus()
+    }
   }
 }
 </script>

src/views/Main/Editor/Tabs/Tab/index.vue

@@ -8,7 +8,7 @@
     >
       <template #left-pane>
         <div class="query-editor">
-          <sql-editor v-model="query" />
+          <sql-editor ref="sqlEditor" v-model="query" />
         </div>
       </template>
       <template #right-pane>
@@ -86,9 +86,6 @@ export default {
       return this.id === this.$store.state.currentTabId
     }
   },
-  created () {
-    this.$store.commit('setCurrentTab', this)
-  },
   mounted () {
     this.resizeObserver = new ResizeObserver(this.handleResize)
     this.resizeObserver.observe(this.$refs.bottomPane)
@@ -98,9 +95,14 @@ export default {
     this.resizeObserver.unobserve(this.$refs.bottomPane)
   },
   watch: {
-    isActive () {
-      if (this.isActive) {
-        this.$store.commit('setCurrentTab', this)
+    isActive: {
+      immediate: true,
+      async handler () {
+        if (this.isActive) {
+          this.$store.commit('setCurrentTab', this)
+          await this.$nextTick()
+          this.$refs.sqlEditor.focus()
+        }
       }
     },
     query () {

tests/lib/utils/fileIo.spec.js

@@ -112,7 +112,7 @@ describe('fileIo.js', () => {
 
     file = { type: 'application/x-sqlite3' }
     expect(fu.isDatabase(file)).to.equal(true)
-    
+
     file = { type: '', name: 'test.db' }
     expect(fu.isDatabase(file)).to.equal(true)
 

tests/views/MainView/Editor/Tabs/Tab/Tab.spec.js

@@ -115,6 +115,7 @@ describe('Tab.vue', () => {
     })
 
     state.currentTabId = 1
+    await wrapper.vm.$nextTick()
     expect(mutations.setCurrentTab.calledOnceWith(state, wrapper.vm)).to.equal(true)
   })