From 32b9c1eba125e02bce50b2e6ad51a3b7195d2469 Mon Sep 17 00:00:00 2001
From: Bram Kragten <mail@bramkragten.nl>
Date: Fri, 18 Sep 2020 12:17:10 +0200
Subject: [PATCH] Fix tap to toggle sidebar (#7049)

---
 src/components/ha-sidebar.ts | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/components/ha-sidebar.ts b/src/components/ha-sidebar.ts
index 0f47cc497a..275c4d5d32 100644
--- a/src/components/ha-sidebar.ts
+++ b/src/components/ha-sidebar.ts
@@ -237,7 +237,7 @@ class HaSidebar extends LitElement {
           ? html`
               <mwc-icon-button
                 .label=${hass.localize("ui.sidebar.sidebar_toggle")}
-                @click=${this._toggleSidebar}
+                @action=${this._toggleSidebar}
               >
                 <ha-svg-icon
                   .path=${hass.dockedSidebar === "docked"
@@ -544,15 +544,12 @@ class HaSidebar extends LitElement {
 
   private async _unhidePanel(ev: Event) {
     ev.preventDefault();
-    const index = this._hiddenPanels.indexOf((ev.currentTarget as any).panel);
-    if (index < 0) {
-      return;
-    }
-    this._hiddenPanels.splice(index, 1);
-    // Make a copy for Memoize
-    this._hiddenPanels = [...this._hiddenPanels];
+    const panel = (ev.currentTarget as any).panel;
     this._renderEmptySortable = true;
     await this.updateComplete;
+    this._hiddenPanels = this._hiddenPanels.filter(
+      (hidden) => hidden !== panel
+    );
     this._renderEmptySortable = false;
   }
 
@@ -648,7 +645,10 @@ class HaSidebar extends LitElement {
     });
   }
 
-  private _toggleSidebar() {
+  private _toggleSidebar(ev: CustomEvent) {
+    if (ev.detail.action !== "tap") {
+      return;
+    }
     fireEvent(this, "hass-toggle-menu");
   }