fixup pat-toolbar: Remove the render URL cleanup method. This is now done directly in pat-base-url.

Author: thet

src/pat/toolbar/toolbar.js

@@ -32,7 +32,7 @@ class Pattern extends BasePattern {
             "navigate",
             "pat-toolbar--history-changed",
             async () => {
-                // Wait a tick to let other Patterns set the baseUrl.
+                // Wait a tick to let other Patterns set the `data-base-url`.
                 await utils.timeout(1);
                 await this.reload_toolbar();
             }
@@ -60,23 +60,11 @@ class Pattern extends BasePattern {
     }
 
     async reload_toolbar() {
-        // Don't reload on content views but on their parent if so.
-        const split_words = [
-            // NOTE: order matters.
-            "@@", // also catches @@folder_contents and @@edit
-            "folder_contents",
-            "edit",
-            "#",
-            "?",
-        ];
+        const render_url = this.options["render-url"];
         let url = document.body.dataset.baseUrl;
-        log.debug("URL before cleanup: ", url);
-        // Split all split words out of url
-        url = split_words.reduce((url_, split_) => url_.split(split_)[0], url);
         // Ensure a trailing slash in the URL.
         url = url[url.length - 1] === "/" ? url : `${url}/`
-        url = `${url}${this.options["render-url"]}`;
-        log.debug("URL after cleanup: ", url);
+        url = `${url}${render_url}`;
 
         if (this.previous_toolbar_url === url) {
             // no need to reload same url
@@ -85,6 +73,7 @@ class Pattern extends BasePattern {
         }
 
         // fetch toolbar
+        log.debug("Reload toolbar on: ", url);
         const response = await fetch(url);
         const data = await response.text();