jekyll-hover-popup 0.2.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '093a8fdc8081648557c5767ccda5e569f65755d513df938c8c96b02bf8653348'
-  data.tar.gz: 293393bc00ed5a9e096fb670b6c1468cf83450403db44f1cdd74258e1e1684c5
+  metadata.gz: d904538318970368b8ae375e4e1296000b2e9a88a25ccbfd7273846cbc42d21b
+  data.tar.gz: e212207697ebb5f6f5a2474078276d242a3b3c6902461473ec9e68f6dac4d818
 SHA512:
-  metadata.gz: 258b38b8e03cdb86f0f1c1ac7c62dc3b56e6046463a28128dd3c7049c0681bd1f0bc609e4a8f357cf3d66c4ca1c8312a6cbc4b72f019fe8392f56ac8c27ff2e6
-  data.tar.gz: '09ab116eea32d0d8f310ea74b75466a19dc6c279ea4b551d22f4cedb8c8574f30c139305433c54283cd131b5291dafa10aea3458abe4165fd8a36ca21ac5dedf'
+  metadata.gz: a1006d757734f03fe07bc223c1feb50ce98e4e557a5f5e4a39cfaba9403865bf1218fb99e332bc7e7cb98bad98e1b084d6dd6ceb5b121ecdcc39c1335ae74e6b
+  data.tar.gz: 8bf2a9bb8af64ababe2c3af6eed644c5ef16a3f2365da453966d0068270cac9c366109c518caad6aedd1b2cddb32671e05c4862e37e81d4ccdf8fe3759fbd83c

data/README.md

@@ -6,14 +6,15 @@ Inspired by the hover window behavior in [5etools](https://github.com/5etools/5e
 
 ## Features
 
-- Hover an internal link to preview its target content in a floating window
-- Hold **Shift** while hovering (or when leaving the link) to pin the preview
-- Pinned windows show **Follow link** and **Close** (Ctrl/Cmd+click Close to close all)
-- Pinned links do not open additional previews on further hovers
-- Section links (`#heading-id`) show only that section; page links show the full page content
-- External links and non-page links are ignored
-- Popup windows are resizable (8 drag handles) and styled from the host page's computed theme
-- Exposes `window.JekyllHoverPopup` for other plugins (for example, image map region clicks)
+- Hover over an internal link to see a reference preview
+- Persist windows by holding the SHIFT key
+- Close all windows by holding CTRL while closing one
+- Minimize windows by double clicking the title bar
+- Navigate to an entry later by clicking the link in the title bar
+- Eight point window resize
+- Exposes `window.JekyllHoverPopup` for other plugins
+
+<img width="640" height="360" alt="window-preview-demo-2" src="https://github.com/user-attachments/assets/1f8f60ad-ad2a-4418-ad0f-0494f4e43faf" />
 
 ## Install
 
@@ -41,10 +42,12 @@ hover_popup:
   enabled: true
   assets_path: /assets/jekyll-hover-popup
   hover_delay_ms: 0
-  nav_hover_preview: true
+  nav_hover_preview: false
 ```
 
-Set `nav_hover_preview: false` to disable hover previews for links inside navigation elements (`.site-nav`, `nav`, `[role="navigation"]`, etc.).
+Add a `hover_delay_ms` to delay the preview window creation
+
+Set `nav_hover_preview: true` to enable hover previews for links inside navigation elements (`.site-nav`, `nav`, `[role="navigation"]`, etc.).
 
 ## JavaScript API
 
@@ -72,4 +75,4 @@ window.JekyllHoverPopup.openContent({
 
 ## License
 
-MIT
+[MIT](LICENSE)

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

@@ -3,7 +3,7 @@
 
   const config = window.__JHP_CONFIG__ || {};
   const HOVER_DELAY_MS = Number(config.hoverDelayMs) || 0;
-  const NAV_HOVER_PREVIEW = config.navHoverPreview !== false;
+  const NAV_HOVER_PREVIEW = config.navHoverPreview === true;
   const TRANSIENT_TITLE_HINT = "SHIFT to persist";
   const MAX_VIEWPORT_WIDTH = 0.92;
   const MAX_VIEWPORT_HEIGHT = 0.92;
@@ -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/hooks.rb

@@ -11,7 +11,7 @@ module Jekyll
           assets_path = cfg["assets_path"] || "/assets/jekyll-hover-popup"
           assets_path = "/#{assets_path}" unless assets_path.start_with?("/")
           hover_delay_ms = cfg["hover_delay_ms"] || 0
-          nav_hover_preview = cfg.fetch("nav_hover_preview", true)
+          nav_hover_preview = cfg.fetch("nav_hover_preview", false)
 
           begin
             doc.output = inject_assets(doc.output.to_s, assets_path: assets_path, hover_delay_ms: hover_delay_ms, nav_hover_preview: nav_hover_preview)
@@ -21,7 +21,7 @@ module Jekyll
         end
       end
 
-      def self.inject_assets(html, assets_path:, hover_delay_ms:, nav_hover_preview: true)
+      def self.inject_assets(html, assets_path:, hover_delay_ms:, nav_hover_preview: false)
         return html if html.include?('data-hover-popup-root="true"')
 
         tags = <<~HTML

data/lib/jekyll/hover_popup/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module HoverPopup
-    VERSION = "0.2.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.2.0
+  version: 0.3.1
 platform: ruby
 authors:
 - directsun