From b4281558b940e1d7648feec3c87cd05c488e8694 Mon Sep 17 00:00:00 2001
From: "Ronald A. Richardson" <me@ron.dev>
Date: Fri, 22 Dec 2023 17:15:29 +0800
Subject: [PATCH 1/4] added sidebar toggle callback function

---
 addon/components/layout/header.hbs               | 2 +-
 addon/components/layout/header/sidebar-toggle.js | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/addon/components/layout/header.hbs b/addon/components/layout/header.hbs
index e533b33..fa251a4 100644
--- a/addon/components/layout/header.hbs
+++ b/addon/components/layout/header.hbs
@@ -4,7 +4,7 @@
             <LogoIcon @brand={{@brand}} @size="8" />
         </LinkToExternal>
         {{#if @sidebarToggleEnabled}}
-            <Layout::Header::SidebarToggle class="mr-2" />
+            <Layout::Header::SidebarToggle class="mr-2" @onToggle={{@onSidebarToggle}} />
         {{/if}}
         {{#unless (media "isMobile")}}
             <div role="menu" class="next-catalog-menu-items flex mr-4">
diff --git a/addon/components/layout/header/sidebar-toggle.js b/addon/components/layout/header/sidebar-toggle.js
index 4b0f8a2..e4c3481 100644
--- a/addon/components/layout/header/sidebar-toggle.js
+++ b/addon/components/layout/header/sidebar-toggle.js
@@ -17,5 +17,9 @@ export default class LayoutHeaderSidebarToggleComponent extends Component {
         }
 
         this.isSidebarVisible = !this.isSidebarVisible;
+
+        if (typeof this.args.onToggle === 'function') {
+            this.args.onToggle(sidebar, this.isSidebarVisible);
+        }
     }
 }

From 6b352450fe744f100f9886bb605e8c02040b4891 Mon Sep 17 00:00:00 2001
From: "Ronald A. Richardson" <me@ron.dev>
Date: Mon, 25 Dec 2023 12:08:02 +0800
Subject: [PATCH 2/4] added disable state to sidebar toggle component

---
 addon/components/layout/header.hbs                | 4 +---
 addon/components/layout/header/sidebar-toggle.hbs | 2 +-
 addon/components/layout/header/sidebar-toggle.js  | 4 ++++
 addon/styles/components/sidebar-toggle.css        | 7 +++++++
 4 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/addon/components/layout/header.hbs b/addon/components/layout/header.hbs
index fa251a4..41665fe 100644
--- a/addon/components/layout/header.hbs
+++ b/addon/components/layout/header.hbs
@@ -3,9 +3,7 @@
         <LinkToExternal @route="console" class="logo navbar-logo mr-4">
             <LogoIcon @brand={{@brand}} @size="8" />
         </LinkToExternal>
-        {{#if @sidebarToggleEnabled}}
-            <Layout::Header::SidebarToggle class="mr-2" @onToggle={{@onSidebarToggle}} />
-        {{/if}}
+        <Layout::Header::SidebarToggle class="mr-2" @onToggle={{@onSidebarToggle}} @disabled={{not @sidebarToggleEnabled}} />
         {{#unless (media "isMobile")}}
             <div role="menu" class="next-catalog-menu-items flex mr-4">
                 {{#each @menuItems as |menuItem|}}
diff --git a/addon/components/layout/header/sidebar-toggle.hbs b/addon/components/layout/header/sidebar-toggle.hbs
index 69d2ad9..ec793ab 100644
--- a/addon/components/layout/header/sidebar-toggle.hbs
+++ b/addon/components/layout/header/sidebar-toggle.hbs
@@ -1,4 +1,4 @@
-<div class="sidebar-toggle-button-wrapper flex-shrink-0 {{if this.isSidebarVisible "sidebar-is-visible"}}" ...attributes>
+<div class="sidebar-toggle-button-wrapper flex-shrink-0 {{if @disabled "disabled opacity-50"}} {{if this.isSidebarVisible "sidebar-is-visible"}}" ...attributes>
     <button type="button" {{on 'click' this.toggleSidebar}} class="sidebar-toggle-button next-org-button-trigger">
         <svg id="sidebar-toggle-icon" data-name="sidebar-toggle-icon" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 392 340">
             <rect x="0" y="0" width="150" height="340" fill={{if this.isSidebarVisible "white" "transparent"}} rx="40" ry="40"></rect>
diff --git a/addon/components/layout/header/sidebar-toggle.js b/addon/components/layout/header/sidebar-toggle.js
index e4c3481..89edd23 100644
--- a/addon/components/layout/header/sidebar-toggle.js
+++ b/addon/components/layout/header/sidebar-toggle.js
@@ -8,6 +8,10 @@ export default class LayoutHeaderSidebarToggleComponent extends Component {
     @tracked isSidebarVisible = true;
 
     @action toggleSidebar() {
+        if (this.args.disabled === true) {
+            return;
+        }
+        
         const sidebar = this.universe.sidebarContext;
 
         if (this.isSidebarVisible) {
diff --git a/addon/styles/components/sidebar-toggle.css b/addon/styles/components/sidebar-toggle.css
index 2b84b5d..8a8c07e 100644
--- a/addon/styles/components/sidebar-toggle.css
+++ b/addon/styles/components/sidebar-toggle.css
@@ -2,6 +2,13 @@
     flex-shrink: 0;
 }
 
+.sidebar-toggle-button-wrapper.disabled:hover,
+.sidebar-toggle-button-wrapper.disabled > button:hover {
+    background-color: transparent;
+    box-shadow: none;
+    border: none;
+}
+
 .sidebar-toggle-button-wrapper > .sidebar-toggle-button.next-org-button-trigger,
 .sidebar-toggle-button-wrapper > .sidebar-toggle-button {
     display: flex;

From f301183b80b3b3f15bb4464c59fcc7a4af45f208 Mon Sep 17 00:00:00 2001
From: "Ronald A. Richardson" <me@ron.dev>
Date: Mon, 25 Dec 2023 12:17:38 +0800
Subject: [PATCH 3/4] made it so sidebar toggle needs `showSidebarToggle`
 option to be explicilty set to render

---
 addon/components/layout/header.hbs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/addon/components/layout/header.hbs b/addon/components/layout/header.hbs
index 41665fe..e373e05 100644
--- a/addon/components/layout/header.hbs
+++ b/addon/components/layout/header.hbs
@@ -3,7 +3,9 @@
         <LinkToExternal @route="console" class="logo navbar-logo mr-4">
             <LogoIcon @brand={{@brand}} @size="8" />
         </LinkToExternal>
-        <Layout::Header::SidebarToggle class="mr-2" @onToggle={{@onSidebarToggle}} @disabled={{not @sidebarToggleEnabled}} />
+        {{#if @showSidebarToggle}}
+            <Layout::Header::SidebarToggle class="mr-2" @onToggle={{@onSidebarToggle}} @disabled={{not @sidebarToggleEnabled}} />
+        {{/if}}
         {{#unless (media "isMobile")}}
             <div role="menu" class="next-catalog-menu-items flex mr-4">
                 {{#each @menuItems as |menuItem|}}

From 899d11cb4dbee0f8dd6f2c38c909b1e30e537c6d Mon Sep 17 00:00:00 2001
From: "Ronald A. Richardson" <me@ron.dev>
Date: Mon, 25 Dec 2023 12:19:22 +0800
Subject: [PATCH 4/4] fixed lint

---
 addon/components/layout/header/sidebar-toggle.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addon/components/layout/header/sidebar-toggle.js b/addon/components/layout/header/sidebar-toggle.js
index 89edd23..4e084d8 100644
--- a/addon/components/layout/header/sidebar-toggle.js
+++ b/addon/components/layout/header/sidebar-toggle.js
@@ -11,7 +11,7 @@ export default class LayoutHeaderSidebarToggleComponent extends Component {
         if (this.args.disabled === true) {
             return;
         }
-        
+
         const sidebar = this.universe.sidebarContext;
 
         if (this.isSidebarVisible) {