From d5c1a5c3f68df034eabf930e8e22ef9527847404 Mon Sep 17 00:00:00 2001 From: Alexandr Stelnykovych Date: Fri, 27 Jun 2025 12:45:58 +0300 Subject: [PATCH] fix(UI): remove search highlight markup from merge profile names https://github.com/safing/portmaster/issues/1490 --- desktop/angular/src/app/pages/app-view/overview.ts | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/desktop/angular/src/app/pages/app-view/overview.ts b/desktop/angular/src/app/pages/app-view/overview.ts index 3c995621..96d8d0e0 100644 --- a/desktop/angular/src/app/pages/app-view/overview.ts +++ b/desktop/angular/src/app/pages/app-view/overview.ts @@ -134,11 +134,22 @@ export class AppOverviewComponent implements OnInit, OnDestroy { }); } + stripHtmlTags(text: string): string { + if (!text) return ''; + // Only strip if we have proper HTML tags (opening and closing with same tag name) + return text.replace(/<([a-zA-Z][a-zA-Z0-9]*)[^>]*>([^<]*)<\/\1>/g, '$2'); + } + openMergeDialog() { this.dialog.create(MergeProfileDialogComponent, { autoclose: true, backdrop: 'light', - data: this.profiles.filter((p) => p.selected), + data: this.profiles.filter((p) => p.selected) + .map((p) => ({ + ...p, + // Strip HTML tags from Name if it exists (e.g., highlighted search results) + ...(p.Name ? { Name: this.stripHtmlTags(p.Name) } : {}) + })), }); this.selectMode = false;