From bfb5a2b041a25236486c947ae157c5cfd098d359 Mon Sep 17 00:00:00 2001
From: Jacob Overgaard <752371+iOvergaard@users.noreply.github.com>
Date: Tue, 11 Apr 2023 15:46:57 +0200
Subject: [PATCH] delete log levels if there are none

---
 .../log-viewer-log-level-filter-menu.element.ts    | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/backoffice/settings/logviewer/workspace/views/search/components/log-viewer-log-level-filter-menu.element.ts b/src/Umbraco.Web.UI.Client/src/backoffice/settings/logviewer/workspace/views/search/components/log-viewer-log-level-filter-menu.element.ts
index 474b64843b..ce2d6eee18 100644
--- a/src/Umbraco.Web.UI.Client/src/backoffice/settings/logviewer/workspace/views/search/components/log-viewer-log-level-filter-menu.element.ts
+++ b/src/Umbraco.Web.UI.Client/src/backoffice/settings/logviewer/workspace/views/search/components/log-viewer-log-level-filter-menu.element.ts
@@ -68,6 +68,8 @@ export class UmbLogViewerLogLevelFilterMenuElement extends UmbLitElement {
 
 		if (logLevels.length) {
 			q = { ...q, loglevels: logLevels.join(',') };
+		} else {
+			delete q.loglevels;
 		}
 
 		window.history.pushState({}, '', `${path()}?${toQueryString(q)}`);
@@ -87,16 +89,16 @@ export class UmbLogViewerLogLevelFilterMenuElement extends UmbLitElement {
 
 	#renderLogLevelSelector() {
 		return html`
-			<div slot="dropdown" id="log-level-selector" @change=${() => this.setLogLevelDebounce()}>
+			<div slot="dropdown" id="log-level-selector" @change=${this.setLogLevelDebounce}>
 				${Object.values(LogLevelModel).map(
 					(logLevel) =>
 						html`<uui-checkbox
 							class="log-level-menu-item"
 							.checked=${this._logLevelFilter.includes(logLevel)}
 							.value=${logLevel}
-							label="${logLevel}"
-							><umb-log-viewer-level-tag .level=${logLevel}></umb-log-viewer-level-tag
-						></uui-checkbox>`
+							label="${logLevel}">
+							<umb-log-viewer-level-tag .level=${logLevel}></umb-log-viewer-level-tag>
+						</uui-checkbox>`
 				)}
 				<uui-button class="log-level-menu-item" @click=${this.#selectAllLogLevels} label="Select all"
 					>Select all</uui-button
@@ -110,8 +112,8 @@ export class UmbLogViewerLogLevelFilterMenuElement extends UmbLitElement {
 
 	render() {
 		return html`
-			<umb-button-with-dropdown label="Select log levels"
-				>Log Level:
+			<umb-button-with-dropdown label="Select log levels">
+				Log Level:
 				${this._logLevelFilter.length > 0
 					? this._logLevelFilter.map((level) => html`<span class="log-level-button-indicator">${level}</span>`)
 					: 'All'}