From 83e934f80c620f7eee65fa7a6f47d23b229b0dc5 Mon Sep 17 00:00:00 2001
From: Ken Sternberg <ken@goauthentik.io>
Date: Fri, 17 Nov 2023 11:18:43 -0800
Subject: [PATCH] Removed the consoe.log

---
 web/src/admin/flows/utils.ts             |  6 +--
 web/src/elements/sidebar/Sidebar.ts      |  8 ++--
 web/src/elements/sidebar/SidebarItems.ts | 56 +++++++++++++++---------
 3 files changed, 44 insertions(+), 26 deletions(-)

diff --git a/web/src/admin/flows/utils.ts b/web/src/admin/flows/utils.ts
index dde724d8c..9dadc3a8b 100644
--- a/web/src/admin/flows/utils.ts
+++ b/web/src/admin/flows/utils.ts
@@ -6,9 +6,9 @@ export function RenderFlowOption(flow: Flow): string {
     return `${flow.slug} (${flow.name})`;
 }
 
-type Pair = [FlowDesignationEnum, string];
+type FlowDesignationPair = [FlowDesignationEnum, string];
 
-export const flowDesignationTable: Pair[] = [
+export const flowDesignationTable: FlowDesignationPair[] = [
     [FlowDesignationEnum.Authentication,        msg("Authentication")],
     [FlowDesignationEnum.Authorization,         msg("Authorization")],
     [FlowDesignationEnum.Enrollment,            msg("Enrollment")],
@@ -25,7 +25,7 @@ export function DesignationToLabel(designation: FlowDesignationEnum): string {
     return flowDesignations.get(designation) ?? msg("Unknown designation");
 }
 
-const layoutToLabel = new Map([
+const layoutToLabel = new Map<LayoutEnum, string>([
     [LayoutEnum.Stacked,               msg("Stacked")],
     [LayoutEnum.ContentLeft,           msg("Content left")],
     [LayoutEnum.ContentRight,          msg("Content right")],
diff --git a/web/src/elements/sidebar/Sidebar.ts b/web/src/elements/sidebar/Sidebar.ts
index 732375560..211825784 100644
--- a/web/src/elements/sidebar/Sidebar.ts
+++ b/web/src/elements/sidebar/Sidebar.ts
@@ -1,7 +1,7 @@
 import { AKElement } from "@goauthentik/elements/Base";
 import "@goauthentik/elements/sidebar/SidebarBrand";
-import "@goauthentik/elements/sidebar/SidebarUser";
 import "@goauthentik/elements/sidebar/SidebarItems";
+import "@goauthentik/elements/sidebar/SidebarUser";
 
 import { CSSResult, TemplateResult, css, html } from "lit";
 import { customElement, property } from "lit/decorators.js";
@@ -73,8 +73,10 @@ export class Sidebar extends AKElement {
     }
 
     render(): TemplateResult {
-        console.log(this.entries);
-        return html`<nav class="pf-c-nav ${this.activeTheme === UiThemeEnum.Light ? "pf-m-light" : ""}" aria-label="Global">
+        return html`<nav
+            class="pf-c-nav ${this.activeTheme === UiThemeEnum.Light ? "pf-m-light" : ""}"
+            aria-label="Global"
+        >
             <ak-sidebar-brand></ak-sidebar-brand>
             <ak-sidebar-items .entries=${this.entries}></ak-sidebar-items>
             <ak-sidebar-user></ak-sidebar-user>
diff --git a/web/src/elements/sidebar/SidebarItems.ts b/web/src/elements/sidebar/SidebarItems.ts
index ba04acb8b..b9a6d7f68 100644
--- a/web/src/elements/sidebar/SidebarItems.ts
+++ b/web/src/elements/sidebar/SidebarItems.ts
@@ -1,4 +1,3 @@
-import { ROUTE_SEPARATOR } from "@goauthentik/common/constants";
 import { AKElement } from "@goauthentik/elements/Base";
 
 import { CSSResult, TemplateResult, css, html, nothing } from "lit";
@@ -25,6 +24,9 @@ export type SidebarAttributes = {
 };
 
 export type SidebarEntry = {
+    // - nullish: This entry is not a link.
+    // - string: the url for the entry
+    // - SidebarEventHandler: a function to run if the entry is clicked.
     path: string | SidebarEventHandler | null;
     label: string;
     attributes?: SidebarAttributes | null; // eslint-disable-line
@@ -34,6 +36,8 @@ export type SidebarEntry = {
 // Typescript requires the type here to correctly type the recursive path
 export type SidebarRenderer = (_: SidebarEntry) => TemplateResult;
 
+const entryKey = (entry: SidebarEntry) => `${entry.path || "no-path"}:${entry.label}`;
+
 @customElement("ak-sidebar-items")
 export class SidebarItems extends AKElement {
     static get styles(): CSSResult[] {
@@ -94,7 +98,7 @@ export class SidebarItems extends AKElement {
                 }
 
                 .pf-c-nav__link a {
-flex: 1 0 max-content;
+                    flex: 1 0 max-content;
                     color: var(--pf-c-nav__link--Color);
                 }
 
@@ -123,12 +127,14 @@ flex: 1 0 max-content;
     @state()
     expanded: WeakSet<SidebarEntry> = new WeakSet();
 
+    current?: SidebarEntry;
+
     constructor() {
         super();
         // this.onToggle = this.onToggle.bind(this);
 
         this.onHashChange = this.onHashChange.bind(this);
-        this.isActive = this.isActive.bind(this);
+        this.recordActive = this.recordActive.bind(this);
         this.renderItem = this.renderItem.bind(this);
         this.toggleExpand = this.toggleExpand.bind(this);
     }
@@ -151,8 +157,8 @@ flex: 1 0 max-content;
         /* no op */
     }
 
-    isActive(path: string) {
-        return true;
+    recordActive(entry: SidebarEntry) {
+        this.current = entry;
     }
 
     render(): TemplateResult {
@@ -180,7 +186,6 @@ flex: 1 0 max-content;
         // not when being forwarded to the correct renderer.
         const attr = attributes ?? undefined;
         const hasChildren = !!(children && children.length > 0);
-        console.log(entry.label, hasChildren);
 
         // This is grossly imperative, in that it HAS to come before the content is rendered
         // to make sure the content gets the right settings with respect to expansion.
@@ -195,8 +200,8 @@ flex: 1 0 max-content;
                 : hasChildren
                 ? this.renderLabelAndChildren(entry)
                 : path
-                ? this.renderLink(label, path, attr)
-                : this.renderLabel(label, attr);
+                ? this.renderLink(entry)
+                : this.renderLabel(entry);
 
         const expanded = {
             "highlighted": !!attr?.highlight,
@@ -207,9 +212,10 @@ flex: 1 0 max-content;
         return html`<li class="pf-c-nav__item ${classMap(expanded)}">${content}</li>`;
     }
 
-    toLinkClasses(attr: SidebarAttributes) {
+    toLinkClasses(entry: SidebarEntry) {
+        const attr = entry.attributes || {};
         return {
-            "pf-m-current": !!attr.isActive,
+            "pf-m-current": this.current === entry,
             "pf-c-nav__link": true,
             "highlight": !!(typeof attr.highlight === "function"
                 ? attr.highlight()
@@ -217,22 +223,29 @@ flex: 1 0 max-content;
         };
     }
 
-    renderLabel(label: string, attr: SidebarAttributes = {}) {
-        return html`<div class=${classMap(this.toLinkClasses(attr))}>${label}</div>`;
+    renderLabel(entry: SidebarEntry) {
+        return html`<div class=${classMap(this.toLinkClasses(entry))}>${entry.label}</div>`;
     }
 
     // note the responsibilities pushed up to the caller
-    renderLink(label: string, path: string | SidebarEventHandler, attr: SidebarAttributes = {}) {
-        if (typeof path === "function") {
-            return html` <a @click=${path} class=${classMap(this.toLinkClasses(attr))}>
-                ${label}
+    renderLink(entry: SidebarEntry) {
+        const record = () => this.recordActive(entry);
+        if (typeof entry.path === "function") {
+            const handler = entry.path;
+            const recordAndHandle = () => {
+                record();
+                handler();
+            };
+            return html` <a @click=${recordAndHandle} class=${classMap(this.toLinkClasses(entry))}>
+                ${entry.label}
             </a>`;
         }
         return html` <a
-            href="${attr.isAbsoluteLink ? "" : "#"}${path}"
-            class=${classMap(this.toLinkClasses(attr))}
+            @click=${record}
+            href="${entry.attributes?.isAbsoluteLink ? "" : "#"}${entry.path}"
+            class=${classMap(this.toLinkClasses(entry))}
         >
-            ${label}
+            ${entry.label}
         </a>`;
     }
 
@@ -259,9 +272,12 @@ flex: 1 0 max-content;
 
     renderLinkAndChildren(entry: SidebarEntry): TemplateResult {
         const handler = () => this.toggleExpand(entry);
-        return html` <div class="pf-c-nav__link">
+        const record = () => this.recordActive(entry);
+        const active = { "pf-m-current": entry === this.current };
+        return html` <div class="pf-c-nav__link ${classMap(active)}">
                 <a
                     href="${entry.attributes?.isAbsoluteLink ? "" : "#"}${entry.path}"
+                    @click=${record}
                     class="ak-nav__link"
                 >
                     ${entry.label}