jekyll-hover-popup 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cdae0cb64015858986ffe6c6dc11d44947b4e172dee08bccd68d191230416b26
-  data.tar.gz: af148150f8a1890b082adbccc2c4665a56dacbad3eea581608dd02c0cbee8f36
+  metadata.gz: d904538318970368b8ae375e4e1296000b2e9a88a25ccbfd7273846cbc42d21b
+  data.tar.gz: e212207697ebb5f6f5a2474078276d242a3b3c6902461473ec9e68f6dac4d818
 SHA512:
-  metadata.gz: a3192b0edd3563b6b267740ca86eff70b040b269d30aff9005366902be46d6f11026644757ea3f010296aa610f91f4969426e82c0bb1860ed473b7ed0cd07f44
-  data.tar.gz: 203ad982a1a4dd62680ee9798179a224835fca23322b2744e1239d09dfcce0ac8a63e63e574f90a58acf893a43cd69204015160ba37d229b2d5d1a9f40076d30
+  metadata.gz: a1006d757734f03fe07bc223c1feb50ce98e4e557a5f5e4a39cfaba9403865bf1218fb99e332bc7e7cb98bad98e1b084d6dd6ceb5b121ecdcc39c1335ae74e6b
+  data.tar.gz: 8bf2a9bb8af64ababe2c3af6eed644c5ef16a3f2365da453966d0068270cac9c366109c518caad6aedd1b2cddb32671e05c4862e37e81d4ccdf8fe3759fbd83c

data/assets/jekyll-hover-popup/hover_popup.js

@@ -128,10 +128,16 @@
     });
   }
 
+  function stripHtmlExtension(pathname) {
+    return pathname.replace(/\.html?$/i, "");
+  }
+
   function normalizePath(pathname) {
     if (!pathname) return "/";
-    const normalized = pathname.endsWith("/") ? pathname : `${pathname}/`;
-    return normalized;
+    // Treat /page.html and /page/ as the same document for cache/same-page checks.
+    let path = stripHtmlExtension(pathname);
+    if (!path.endsWith("/")) path = `${path}/`;
+    return path || "/";
   }
 
   function normalizeLinkKey(url) {
@@ -145,6 +151,30 @@
     );
   }
 
+  // Prefer paths that static hosts (esp. Cloudflare Pages) resolve correctly.
+  // Never request `page.html/` — that often SPA-fallbacks to the site index.
+  function fetchPathCandidates(pathname) {
+    const paths = [];
+    const push = (p) => {
+      if (p && !paths.includes(p)) paths.push(p);
+    };
+
+    if (/\.html?$/i.test(pathname)) {
+      push(pathname);
+      const withoutExt = stripHtmlExtension(pathname) || "/";
+      push(withoutExt);
+      if (!withoutExt.endsWith("/")) push(`${withoutExt}/`);
+      return paths;
+    }
+
+    const withSlash = pathname.endsWith("/") ? pathname : `${pathname}/`;
+    const withoutSlash = pathname.endsWith("/") ? pathname.replace(/\/+$/, "") || "/" : pathname;
+    push(withSlash);
+    push(withoutSlash);
+    if (withoutSlash !== "/") push(`${withoutSlash}.html`);
+    return paths;
+  }
+
   function isImagePath(pathname) {
     return /\.(avif|bmp|gif|ico|jpe?g|png|svg|webp)$/i.test(pathname);
   }
@@ -281,30 +311,25 @@
   }
 
   async function fetchPageDocument(pathname) {
-    const cacheKey = pathname.endsWith("/") ? pathname : `${pathname}/`;
+    const cacheKey = normalizePath(pathname);
     if (pageCache.has(cacheKey)) return pageCache.get(cacheKey);
 
-    const fetchPath = cacheKey;
-    const response = await fetch(fetchPath, { credentials: "same-origin" });
-    if (!response.ok) {
-      const fallback = pathname.endsWith("/") ? pathname.slice(0, -1) : `${pathname}/`;
-      if (fallback !== fetchPath) {
-        const retry = await fetch(fallback, { credentials: "same-origin" });
-        if (retry.ok) {
-          const html = await retry.text();
-          const doc = new DOMParser().parseFromString(html, "text/html");
-          pageCache.set(cacheKey, doc);
-          pageCache.set(fallback, doc);
-          return doc;
-        }
+    let lastError;
+    for (const fetchPath of fetchPathCandidates(pathname)) {
+      try {
+        const response = await fetch(fetchPath, { credentials: "same-origin" });
+        if (!response.ok) continue;
+
+        const html = await response.text();
+        const doc = new DOMParser().parseFromString(html, "text/html");
+        pageCache.set(cacheKey, doc);
+        return doc;
+      } catch (err) {
+        lastError = err;
       }
-      throw new Error(`Failed to load ${fetchPath}`);
     }
 
-    const html = await response.text();
-    const doc = new DOMParser().parseFromString(html, "text/html");
-    pageCache.set(cacheKey, doc);
-    return doc;
+    throw lastError || new Error(`Failed to load ${pathname}`);
   }
 
   async function loadLinkContent(anchor) {
@@ -312,7 +337,7 @@
     const hash = url.hash;
 
     let doc;
-    if (url.pathname === window.location.pathname && url.search === window.location.search) {
+    if (isSamePage(url)) {
       doc = document;
     } else {
       doc = await fetchPageDocument(url.pathname);

data/lib/jekyll/hover_popup/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module HoverPopup
-    VERSION = "0.3.0"
+    VERSION = "0.3.1"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-hover-popup
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - directsun