(kb) only activate kb nav when actually using the keyboard

Just trying to disable the mouse-specific stuff to see if tests pass or
not right now.

Author: manuhabitela

app/client/components/RegionFocusSwitcher.ts

@@ -94,11 +94,11 @@ export class RegionFocusSwitcher extends Disposable {
     this._app.on('clipboard_focus', focusActiveSection);
     this.onDispose(() => this._app.off('clipboard_focus', focusActiveSection));
 
-    if (this._gristDocObs) {
+    /*if (this._gristDocObs) {
       const onClick = this._onClick.bind(this);
       pageContainer.addEventListener('mouseup', onClick);
       this.onDispose(() => pageContainer.removeEventListener('mouseup', onClick));
-    }
+    }*/
 
     this._initiated.set(true);
   }
@@ -216,7 +216,7 @@ export class RegionFocusSwitcher extends Disposable {
    *   - if necessary, make it easier to tab through things inside panels by "unfocusing" the view section,
    *   - make sure the internal current region info is set when user clicks on the view layout.
    */
-  private _onClick(event: MouseEvent) {
+  /*private _onClick(event: MouseEvent) {
     const gristDoc = this._getGristDoc();
     if (!gristDoc) {
       return;
@@ -259,7 +259,7 @@ export class RegionFocusSwitcher extends Disposable {
     if (!targetsMain && getPanelElement(targetRegionId as Panel) === event.target) {
       focusPanel();
     }
-  }
+  }*/
 
   private _onEscapeKeypress() {
     const {region: current, initiator} = this._state.get();
@@ -558,7 +558,7 @@ const getPanelElementId = (id: Panel): string => {
   return `[${ATTRS.regionId}="${id}"]`;
 };
 
-const isFocusableElement = (el: EventTarget | null): boolean => {
+/*const isFocusableElement = (el: EventTarget | null): boolean => {
   if (!el) {
     return false;
   }
@@ -569,7 +569,7 @@ const isFocusableElement = (el: EventTarget | null): boolean => {
     return true;
   }
   return false;
-};
+};*/
 
 /**
  * Remove the visual highlight on elements that are styled as focused elements of panels.

test/nbrowser/RegionFocusSwitcher.ts

@@ -253,7 +253,7 @@ describe("RegionFocusSwitcher", function () {
     assert.isTrue(await isClipboardFocused());
   });
 
-  it("should focus a panel-region when clicking an input child element", async function() {
+  it.skip("should focus a panel-region when clicking an input child element", async function() {
     const session = await gu.session().teamSite.login();
     await session.tempNewDoc(cleanup);
 
@@ -268,7 +268,7 @@ describe("RegionFocusSwitcher", function () {
     assert.isTrue(await isClipboardFocused());
   });
 
-  it("should focus a section-region when clicking on it", async function() {
+  it.skip("should focus a section-region when clicking on it", async function() {
     const session = await gu.session().teamSite.login();
     await session.tempNewDoc(cleanup);