Logic should be in utils, use new URL

Author: suprith-hub

frontend/src/static/js/components/webstatus-overview-filters.ts

@@ -230,13 +230,10 @@ export class WebstatusOverviewFilters extends LitElement {
   };
 
   gotoFilterQueryString(): void {
-    const newUrl = `${window.location.origin}${formatOverviewPageUrl(
-      this.location,
-      {
-        q: (this.typeaheadRef.value as WebstatusTypeahead).value,
-        start: 0,
-      },
-    )}`;
+    const newUrl = formatOverviewPageUrl(this.location, {
+      q: (this.typeaheadRef.value as WebstatusTypeahead).value,
+      start: 0,
+    });
     navigateToUrl(newUrl);
   }
 

frontend/src/static/js/utils/app-router.ts

@@ -55,8 +55,11 @@ export const navigateToUrl = (url: string, event?: MouseEvent) => {
     return;
   }
 
+  // Construct a full URL to handle relative paths correctly
+  const fullNewUrl = new URL(url, window.location.origin).href; // TODO for future - handle try/catch case until we use AppLocation
+
   const currentUrl = window.location.href;
-  if (currentUrl === url) {
+  if (currentUrl === fullNewUrl) {
     return;
   }