From d84d1fc55d73458b10aea6be63491cb48ebef22d Mon Sep 17 00:00:00 2001
From: Mads Rasmussen <madsr@hey.com>
Date: Sun, 28 Apr 2024 13:11:40 +0200
Subject: [PATCH] only render context menu when it is open

---
 .../section-sidebar.element.ts                | 27 +++++++++++++------
 1 file changed, 19 insertions(+), 8 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/packages/core/section/section-sidebar/section-sidebar.element.ts b/src/Umbraco.Web.UI.Client/src/packages/core/section/section-sidebar/section-sidebar.element.ts
index 65da98a094..c330a5a1dd 100644
--- a/src/Umbraco.Web.UI.Client/src/packages/core/section/section-sidebar/section-sidebar.element.ts
+++ b/src/Umbraco.Web.UI.Client/src/packages/core/section/section-sidebar/section-sidebar.element.ts
@@ -1,20 +1,31 @@
 import { UmbSectionSidebarContext } from './section-sidebar.context.js';
 import { UmbTextStyles } from '@umbraco-cms/backoffice/style';
-import { css, html, customElement } from '@umbraco-cms/backoffice/external/lit';
+import { css, html, customElement, state, nothing } from '@umbraco-cms/backoffice/external/lit';
 import { UmbLitElement } from '@umbraco-cms/backoffice/lit-element';
 
 @customElement('umb-section-sidebar')
 export class UmbSectionSidebarElement extends UmbLitElement {
+	@state()
+	private _isOpen = false;
+
 	#sectionSidebarContext = new UmbSectionSidebarContext(this);
 
+	constructor() {
+		super();
+
+		this.observe(this.#sectionSidebarContext.contextMenuIsOpen, (value) => {
+			this._isOpen = value;
+		});
+	}
+
 	render() {
-		return html`
-			<umb-section-sidebar-context-menu>
-				<uui-scroll-container id="scroll-container">
-					<slot></slot>
-				</uui-scroll-container>
-			</umb-section-sidebar-context-menu>
-		`;
+		return this._isOpen
+			? html` <umb-section-sidebar-context-menu> ${this.#renderScrollContainer()} </umb-section-sidebar-context-menu> `
+			: this.#renderScrollContainer();
+	}
+
+	#renderScrollContainer() {
+		return html` <uui-scroll-container id="scroll-container"> <slot></slot> </uui-scroll-container> `;
 	}
 
 	static styles = [