From 9eb5c425dc398c20d9f93f2dfd10d0253721ebd7 Mon Sep 17 00:00:00 2001
From: Mads Rasmussen <madsr@hey.com>
Date: Fri, 6 Jan 2023 11:41:38 +0100
Subject: [PATCH] clean up tree navigator to load tree root when inserted

---
 .../tree/navigator/tree-navigator.element.ts  | 98 +------------------
 1 file changed, 3 insertions(+), 95 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/backoffice/shared/components/tree/navigator/tree-navigator.element.ts b/src/Umbraco.Web.UI.Client/src/backoffice/shared/components/tree/navigator/tree-navigator.element.ts
index d232ead8b9..9eb80fbb5a 100644
--- a/src/Umbraco.Web.UI.Client/src/backoffice/shared/components/tree/navigator/tree-navigator.element.ts
+++ b/src/Umbraco.Web.UI.Client/src/backoffice/shared/components/tree/navigator/tree-navigator.element.ts
@@ -1,12 +1,8 @@
 import { css, html } from 'lit';
 import { repeat } from 'lit/directives/repeat.js';
 import { UUITextStyles } from '@umbraco-ui/uui-css/lib';
-import { customElement, property, state } from 'lit/decorators.js';
-import { ifDefined } from 'lit-html/directives/if-defined.js';
+import { customElement, state } from 'lit/decorators.js';
 import { UmbLitElement } from '@umbraco-cms/element';
-import { UmbSectionContext } from '../../section/section.context';
-import { UmbTreeContextMenuService } from '../context-menu/tree-context-menu.service';
-import { UmbTreeContext } from '../tree.context';
 import type { Entity } from '@umbraco-cms/models';
 import { UmbTreeDataStore } from '@umbraco-cms/stores/store';
 import { DocumentTreeItem } from '@umbraco-cms/backend-api';
@@ -17,33 +13,13 @@ import '../tree-item.element';
 export class UmbTreeNavigator extends UmbLitElement {
 	static styles = [UUITextStyles, css``];
 
-	private _storeAlias = '';
-	@property({ attribute: 'store-context-alias' })
-	public get storeAlias() {
-		return this._storeAlias;
-	}
-
-	public set storeAlias(value) {
-		this._storeAlias = value;
-		this._provideStoreContext();
-	}
-
 	@state()
 	private _loading = true;
 
 	@state()
 	private _items: DocumentTreeItem[] = [];
 
-	@state()
-	private _href?: string;
-
-	@state()
-	private _treeRootNode: any;
-
-	private _treeContext?: UmbTreeContext;
-	private _treeContextMenuService?: UmbTreeContextMenuService;
 	private _store?: UmbTreeDataStore<DocumentTreeItem>;
-	private _sectionContext?: UmbSectionContext;
 
 	constructor() {
 		super();
@@ -51,39 +27,10 @@ export class UmbTreeNavigator extends UmbLitElement {
 		this.consumeContext('umbStore', (store) => {
 			this._store = store;
 		});
-
-		this.consumeContext('umbTreeContext', (treeContext: UmbTreeContext) => {
-			this._treeContext = treeContext;
-
-			this._treeRootNode = {
-				name: treeContext.tree.meta.label || '',
-				icon: treeContext.tree.meta.icon || '',
-				type: treeContext.tree.meta.rootNodeEntityType,
-				hasChildren: true,
-				parentKey: null,
-			};
-		});
-
-		this.consumeContext('umbTreeContextMenuService', (treeContextMenuService: UmbTreeContextMenuService) => {
-			this._treeContextMenuService = treeContextMenuService;
-		});
-
-		this.consumeContext('umbSectionContext', (sectionContext: UmbSectionContext) => {
-			this._sectionContext = sectionContext;
-			this._observeSection();
-		});
 	}
 
-	private _provideStoreContext() {
-		if (!this._storeAlias) return;
-
-		this.consumeContext(this._storeAlias, (store) => {
-			this._store = store;
-			this.provideContext('umbStore', store);
-		});
-	}
-
-	private _onShowRoot() {
+	connectedCallback() {
+		super.connectedCallback();
 		this._observeTreeRoot();
 	}
 
@@ -99,46 +46,7 @@ export class UmbTreeNavigator extends UmbLitElement {
 		});
 	}
 
-	private _observeSection() {
-		if (!this._sectionContext) return;
-
-		this.observe(this._sectionContext?.data, (section) => {
-			this._href = this._constructPath(section.meta.pathname, this._treeRootNode?.type);
-		});
-	}
-
-	// TODO: how do we handle this?
-	private _constructPath(sectionPathname: string, type: string | undefined) {
-		return type ? `section/${sectionPathname}/${type}` : undefined;
-	}
-
-	private _openActions() {
-		if (!this._treeContext || !this._sectionContext || !this._treeRootNode) return;
-
-		this._sectionContext?.setActiveTree(this._treeContext?.tree);
-		this._sectionContext?.setActiveTreeItem(this._treeRootNode);
-		this._treeContextMenuService?.open({ name: this._treeRootNode.name, key: this._treeRootNode.key });
-	}
-
 	render() {
-		// TODO: can we use tree item here instead so we don't have duplicated code?
-		// TODO: how do we know if a tree has children?
-		return html` <uui-menu-item
-			label="${ifDefined(this._treeRootNode?.name)}"
-			@show-children=${this._onShowRoot}
-			href="${ifDefined(this._href)}"
-			has-children>
-			${this._renderRootItems()}
-			<uui-icon slot="icon" name="${ifDefined(this._treeRootNode?.icon)}"></uui-icon>
-			<uui-action-bar slot="actions">
-				<uui-button @click=${this._openActions} label="Open actions menu">
-					<uui-symbol-more></uui-symbol-more>
-				</uui-button>
-			</uui-action-bar>
-		</uui-menu-item>`;
-	}
-
-	private _renderRootItems() {
 		// TODO: Fix Type Mismatch ` as Entity` in this template:
 		return html`
 			${repeat(