Fixing JS

Author: glenrobson

3d-guide/README.md

@@ -71,30 +71,30 @@ Now we have the URL to the manifest we can supply this to a number of IIIF viewe
 
 <div class="row">
     <label for="text">Manifest URL:</label>
-    <input id="text" type="text" placeholder="e.g. https://api.npoint.io/8af5347d219a3cec6039" />
+    <input id="text" type="text" minlength="50" placeholder="e.g. https://api.npoint.io/8af5347d219a3cec6039" />
   </div>
 
   <div class="row" style="margin-top:.75rem;">
     <label for="dest">IIIF Viewer</label>
     <select id="dest">
       <!-- Append as query: ?q=TEXT -->
-      <option value="https://www.morphosource.org/uv/uv.html#" data-mode="query" data-param="manifest">
+      <option value="https://www.morphosource.org/uv/uv.html#" data-mode="append" data-param="manifest">
          MorphoSource
       </option>
       <!-- Append as path: /wiki/TEXT -->
-      <option value="https://smithsonian.github.io/voyager-dev/iiif/iiif_demo" data-mode="query" data-param="document">
+      <option value="https://smithsonian.github.io/voyager-dev/iiif/iiif_demo" data-mode="append" data-param="document">
         Voyager
       </option>
       <!-- Another path example: /users/TEXT -->
-      <option value="https://uv-v4.netlify.app/#"  data-mode="query" data-param="manifest">
+      <option value="https://uv-v4.netlify.app/#"  data-mode="append" data-param="manifest">
         Universal Viewer 
       </option>
-      <option value="https://spri-open-resources.s3.us-east-2.amazonaws.com/iiif3dtsg/manifest/index.html#" data-mode="query" data-param="manifest">
+      <option value="https://spri-open-resources.s3.us-east-2.amazonaws.com/iiif3dtsg/manifest/index.html#" data-mode="append" data-param="manifest">
         X3DOM Viewer
       </option>
-      <option value="https://blacklodge.hki.uni-koeln.de/viewer/?locale=en&standalone=true"  data-mode="query" data-param="manifest">
+     <!-- <option value="https://blacklodge.hki.uni-koeln.de/viewer/?locale=en&standalone=true"  data-mode="query" data-param="manifest">
         Kompakkt Viewer
-      </option>
+      </option>-->
     </select>
 
     <button id="go" type="button">Open</button>
@@ -108,23 +108,23 @@ Now we have the URL to the manifest we can supply this to a number of IIIF viewe
       const sel   = $("dest");
       const base  = sel.value;
       const mode  = sel.options[sel.selectedIndex].dataset.mode;
-      const param = sel.options[sel.selectedIndex].dataset.param || "q";
+      const param = sel.options[sel.selectedIndex].dataset.param;
 
       // Build the final URL
       let urlStr;
       try {
-        const u = new URL(base, window.location.href); // supports absolute or relative base
-
-        if (mode === "path") {
-          // Ensure exactly one trailing slash before appending, then encode the segment
-          const needsSlash = u.pathname.endsWith("/") ? "" : "/";
-          u.pathname = u.pathname + needsSlash + encodeURIComponent(input);
+        if (mode === "append") {
+            urlStr = base + "?" + param + "=" + input;
         } else {
-          // Query mode (e.g., ?q=TEXT). If input empty, we still open base.
-          if (input) u.searchParams.set(param, input);
+            const u = new URL(base, window.location.href); // supports absolute or relative base
+            // Query mode (e.g., ?q=TEXT). If input empty, we still open base.
+            if (input) {
+                u.searchParams.set(param, input);
+            }
+
+            urlStr = u.toString();
         }
 
-        urlStr = u.toString();
       } catch (e) {
         // Fallback if base isn't a valid URL for some reason
         urlStr = base + encodeURIComponent(input);