From 7ab07c69aaebf7894d671a7a38c80f2b0f7dfc1b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kristijan=20Ribari=C4=87?= <kriba13@gmail.com>
Date: Sat, 9 Nov 2024 17:18:01 +0100
Subject: [PATCH 1/2] Add workspace navigation via mouse back/forward buttons

Adds the ability to navigate workspaces using the mouse
back/forward buttons when hovering over the sidebar.  This
improves the user experience by providing a more intuitive
way to switch between workspaces.

This feature is only enabled when workspaces are enabled.
It intercepts the "AppCommand" events to handle back/forward
navigation.
---
 .../base/zen-components/ZenWorkspaces.mjs     | 49 +++++++++++++++++++
 1 file changed, 49 insertions(+)

diff --git a/src/browser/base/zen-components/ZenWorkspaces.mjs b/src/browser/base/zen-components/ZenWorkspaces.mjs
index b1280f75b..485ef39cf 100644
--- a/src/browser/base/zen-components/ZenWorkspaces.mjs
+++ b/src/browser/base/zen-components/ZenWorkspaces.mjs
@@ -11,6 +11,7 @@ var ZenWorkspaces = new (class extends ZenMultiWindowFeature {
     cumulativeDelta: 0,
     direction: null
   };
+  _hoveringSidebar = false;
 
   async init() {
     if (!this.shouldHaveWorkspaces) {
@@ -47,10 +48,58 @@ var ZenWorkspaces = new (class extends ZenMultiWindowFeature {
     console.info('ZenWorkspaces: ZenWorkspaces initialized');
 
     this.initializeGestureHandlers();
+    this.initializeWorkspaceNavigation();
 
     Services.obs.addObserver(this, 'weave:engine:sync:finish');
   }
 
+  initializeWorkspaceNavigation() {
+    this._setupAppCommandHandlers();
+    this._setupHoverDetection();
+  }
+
+  _setupAppCommandHandlers() {
+    // Remove existing handler temporarily - this is needed so that _handleAppCommand is called before the original
+    window.removeEventListener("AppCommand", HandleAppCommandEvent, true);
+
+    // Add our handler first
+    window.addEventListener("AppCommand", this._handleAppCommand.bind(this), true);
+
+    // Re-add original handler
+    window.addEventListener("AppCommand", HandleAppCommandEvent, true);
+  }
+
+  _handleAppCommand(event) {
+    if (!this.workspaceEnabled || !this._hoveringSidebar) {
+      return;
+    }
+
+    switch (event.command) {
+      case "Forward":
+        this.changeWorkspaceShortcut(1);
+        event.stopImmediatePropagation();
+        event.preventDefault();
+        break;
+      case "Back":
+        this.changeWorkspaceShortcut(-1);
+        event.stopImmediatePropagation();
+        event.preventDefault();
+        break;
+    }
+  }
+
+  _setupHoverDetection() {
+    const toolbox = document.getElementById('navigator-toolbox');
+
+    toolbox.addEventListener('mouseenter', () => {
+      this._hoveringSidebar = true;
+    });
+
+    toolbox.addEventListener('mouseleave', () => {
+      this._hoveringSidebar = false;
+    });
+  }
+
   initializeGestureHandlers() {
     const elements = [
       document.getElementById('navigator-toolbox'),

From 14b8a1d8a2514812d5571cc804e8378be0434a61 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kristijan=20Ribari=C4=87?= <kriba13@gmail.com>
Date: Sat, 9 Nov 2024 17:20:04 +0100
Subject: [PATCH 2/2] (refactor) Remove debug logging

Removed a debug `console.log` statement that was used
during swipe gesture implementation.  This is no
longer needed.
---
 src/browser/base/zen-components/ZenWorkspaces.mjs | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/browser/base/zen-components/ZenWorkspaces.mjs b/src/browser/base/zen-components/ZenWorkspaces.mjs
index 485ef39cf..c6a4bb181 100644
--- a/src/browser/base/zen-components/ZenWorkspaces.mjs
+++ b/src/browser/base/zen-components/ZenWorkspaces.mjs
@@ -162,7 +162,6 @@ var ZenWorkspaces = new (class extends ZenMultiWindowFeature {
       this._swipeState.direction = this._swipeState.cumulativeDelta > 0 ? 'right' : 'left';
     }
 
-    console.log('MozSwipeGestureUpdateEND', this._swipeState);
   }
 
   async _handleSwipeEnd(event) {